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

Wait for guest connection before test restart #3291

Merged

Conversation

happz
Copy link
Collaborator

@happz happz commented Oct 15, 2024

Even without a reboot, tmt still needs to verify the guest is up and running. The reboot might be triggered beyond the control of tmt, and that is fine, we just need to be sure we restart the test on guest that's alive.

Related to #3284

Pull Request Checklist

  • implement the feature

@happz happz added the bug Something isn't working label Oct 15, 2024
@happz happz added this to the 1.38 milestone Oct 15, 2024
@happz happz added the ci | full test Pull request is ready for the full test execution label Oct 16, 2024
@happz happz force-pushed the test-restart-without-reboot-must-still-wait-for-reconnect branch from 96b27f9 to 8036ca5 Compare October 16, 2024 13:19
Copy link
Collaborator

@skycastlelily skycastlelily left a comment

Choose a reason for hiding this comment

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

LBTM^^

@happz happz force-pushed the test-restart-without-reboot-must-still-wait-for-reconnect branch 2 times, most recently from 7ba95f2 to 7cd12f2 Compare October 22, 2024 07:15
Even without a reboot, tmt still needs to verify the guest is up and
running. The reboot might be triggered beyond the control of tmt, and
that is fine, we just need to be sure we restart the test on guest
that's alive.

Related to #3284
@psss psss force-pushed the test-restart-without-reboot-must-still-wait-for-reconnect branch from 7cd12f2 to 80a0ae1 Compare October 22, 2024 12:17
@psss psss added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label Oct 22, 2024
@psss psss merged commit d9f1430 into main Oct 22, 2024
22 checks passed
@psss psss deleted the test-restart-without-reboot-must-still-wait-for-reconnect branch October 22, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci | full test Pull request is ready for the full test execution status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants