-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Modify kubevirt_vm crud/wait logic #54404
Conversation
7c10e45
to
f1b80fc
Compare
@machacekondra @pkliczewski – our e2e testing works ok: https://travis-ci.org/kubevirt/ansible-kubevirt-modules/jobs/511551711 |
0311893
to
99688c5
Compare
99688c5
to
80002b7
Compare
80002b7
to
f193b13
Compare
f193b13
to
c8f9730
Compare
And done. As far as I can tell everything is working correctly. I'll add a PR extending our travis playbooks to make sure all of this always gets solidly tested. Other than that, once the ansible CI finishes, you can shipit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
d2003e9
to
e496330
Compare
Right, turns out there were still a few corner cases I've missed. Should be ready in a few hours. |
bc5a2a6
to
af65797
Compare
And done. I'll go write a playbook for our repo to make sure all the corner cases I currently have in my head are actually properly tested and verified working |
👍 |
Ok, so kubevirt/ansible-kubevirt-modules#218 contains a comprehensive list of all the stuff this code does and tests for whether it does them correctly. That whole playbook passes on my system. So as far as I can tell this code is correct. Please add shipits and let's wait for someone to merge. |
+1 |
@mmazur Can you please rebase looks like one of the PRs I've just merged has caused a conflict |
1. Adds proper wait support for VM stops and starts 2. Detect kubevirt/ansible-kubevirt-modules#177 and return a sane error 3. Switch to openshift-restclient 0.9.x style wait code
af65797
to
6949ac2
Compare
@machacekondra @pkliczewski had to rebase, please +1 again |
+1 |
Thanks for the quick rebase. Merged into |
SUMMARY
ISSUE TYPE
COMPONENT NAME
kubevirt/kubevirt_vm