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

Try unskipping the previously flakey libvirt tests to check them #2183

Merged

Conversation

stevenhorsman
Copy link
Member

Work through the skipped libvirt tests to see if they are more stable now...

@stevenhorsman stevenhorsman added test_e2e_libvirt Run Libvirt e2e tests test_e2e_libvirt_amd64 and removed test_e2e_libvirt Run Libvirt e2e tests labels Dec 4, 2024
@stevenhorsman stevenhorsman reopened this Dec 5, 2024
@stevenhorsman stevenhorsman added test_e2e_libvirt Run Libvirt e2e tests and removed test_e2e_libvirt_amd64 labels Dec 5, 2024
@stevenhorsman stevenhorsman force-pushed the unskip-libvirt-tests branch 5 times, most recently from e2107a8 to 179202d Compare December 10, 2024 13:01
Unskipping the previously flakey libvirt tests. We've had some code
fixes in kata and environmental changes that might have helped
improve the stability of these tests.

Signed-off-by: stevenhorsman <[email protected]>
@stevenhorsman stevenhorsman changed the title WIP: Try unskipping the previously flakey libvirt tests to check them Try unskipping the previously flakey libvirt tests to check them Dec 11, 2024
@stevenhorsman stevenhorsman marked this pull request as ready for review December 11, 2024 13:28
@stevenhorsman stevenhorsman requested a review from a team as a code owner December 11, 2024 13:28
@stevenhorsman
Copy link
Member Author

Over the course of testing and rebasing this PR I think I've run the e2e tests around 10 times and other than the couple of tests that I left skipped I haven't seen any failures. Of course I might have got lucky, but I think it's worth re-enabling these if they are stable now.

@stevenhorsman stevenhorsman merged commit b9679f0 into confidential-containers:main Dec 11, 2024
40 checks passed
@stevenhorsman
Copy link
Member Author

Merged, lets see how this goes 🤞

@stevenhorsman stevenhorsman deleted the unskip-libvirt-tests branch December 11, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test_e2e_libvirt Run Libvirt e2e tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants