Skip to content

Fix VM wait_for_status calls #937

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.

Sign up for GitHub

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

Jump to bottom
Merged
rnetser merged 1 commit into from
Nov 2, 2022
Merged

Fix VM wait_for_status calls #937

rnetser merged 1 commit into from
Nov 2, 2022

Conversation

@myakove
Copy link
Collaborator

@myakove myakove commented Nov 2, 2022

Need to call wait_for_ready_status to get the VM status.

@cnv-qe-bot cnv-qe-bot requested a review from rnetser November 2, 2022 14:14
@cnv-qe-bot cnv-qe-bot mentioned this pull request Nov 2, 2022
Closed
@cnv-qe-bot
Copy link

cnv-qe-bot commented Nov 2, 2022

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR.

Available user actions:

  • To mark PR as verified add !verified to a PR comment, to un-verify add !-verified to a PR comment.
    Verified label removed on each new commit push.
  • To cherry pick a merged PR add !cherry-pick <target branch to cherry-pick to> to a PR comment.

@myakove
Copy link
Collaborator Author

myakove commented Nov 2, 2022

!verified

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@rnetser rnetser merged commit 54a5b28 into RedHatQE:main Nov 2, 2022
@myakove
Copy link
Collaborator Author

myakove commented Nov 2, 2022

!cherry-pick v4.12

@myakove myakove deleted the vm-fix-wait_for_status branch November 2, 2022 14:22
@cnv-qe-bot cnv-qe-bot mentioned this pull request Nov 2, 2022
Merged
cnv-qe-bot pushed a commit that referenced this pull request Nov 2, 2022
@myakove
Fix VM wait_for_status calls (#937)
4de4323
@cnv-qe-bot
Copy link

cnv-qe-bot commented Nov 2, 2022

Cherry-picked PR Fix VM wait_for_status calls into v4.12

myakove added a commit that referenced this pull request Nov 2, 2022
@cnv-qe-bot @myakove
Fix VM wait_for_status calls (#937) (#939)
d67b141 
Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
cnv-qe-bot added a commit that referenced this pull request Nov 2, 2022
@cnv-qe-bot @myakove
Fix VM wait_for_status calls (#937) (#939)
c65443f 
Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
myakove added a commit that referenced this pull request Nov 2, 2022
@cnv-qe-bot @myakove
Fix VM wait_for_status calls (#937) (#939) (#941)
0dfcfbc 
Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>

Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
myakove added a commit that referenced this pull request Aug 11, 2025
@myakove
Fix VM wait_for_status calls (#937)
11ef2da
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rnetser rnetser rnetser approved these changes

Assignees

@myakove myakove

Labels

Approved-by-rnetser branch-main size/XS verified

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@myakove @cnv-qe-bot @rnetser