Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: use podman instead of ssh to speed up in PR CI #7489

Closed
wants to merge 2 commits into from

Conversation

pbrezina
Copy link
Member

We can now use podman instead of ssh run commands on the host. This is
quite faster so we can benefit from it in PR CI.

Depends on:

@jakub-vavra-cz
Copy link
Contributor

This PR misses backport labels, please make sure to add them.

Copy link
Contributor

@justin-stephenson justin-stephenson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack, thank you.

Copy link
Contributor

@thalman thalman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, thanks

Latest version added an option to replace SSH connections with podman
or docker, therefore a generic interface was created. Most notably,
`host.ssh` was replaced with `host.conn`.
We can now use podman instead of ssh run commands on the host. This
is quite faster so we can benefit from it in PR CI.
@pbrezina
Copy link
Member Author

pbrezina commented Aug 2, 2024

Rebased, let's see if we can get green tests.

@pbrezina
Copy link
Member Author

pbrezina commented Aug 2, 2024

Failures are unrelated.

@pbrezina pbrezina added Ready to push Ready to push no-backport This should go to target branch only. and removed Ready to push Ready to push branch: sssd-2-8 backport-to-stable labels Aug 2, 2024
@pbrezina
Copy link
Member Author

pbrezina commented Aug 2, 2024

Pushed PR: #7489

  • master
    • 8e59f77 - tests: use podman instead of ssh to speed up in PR CI
    • d213e59 - tests: update the tests to work with latest pytest-mh

@pbrezina pbrezina added Pushed and removed Accepted Ready to push Ready to push labels Aug 2, 2024
@pbrezina pbrezina closed this Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked no-backport This should go to target branch only. Pushed Tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants