-
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
bump version of 'vcenter-test-container' to '1.3.0' (govmomi v0.18.0) #41151
Conversation
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
@@ -48,4 +48,4 @@ | |||
- name: make sure no changes were made | |||
assert: | |||
that: | |||
- "poweroff_d1_c1_f0.results|map(attribute='changed')|unique|list == [False]" | |||
- "poweroff_d1_c1_f0.results|map(attribute='changed')|unique|list == [True]" |
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.
Why is the task name make sure no changes were made
when we're asserting that changes were made?
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.
In previous VCSIM version, vms were spawned in "poweredoff" state, and running this PB would poweroff them. But now with latest changes in VCSIM, vms are spawned in "PoweredOn" mode so running the PB would make "changed" as True. I agree to change task name as it is confusing.
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.
@Akasurde Thanks for the explanation.
@dericcrago This PR should be ready to merge once the task names are fixed.
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.
…ansible#41151) * leave vmware_guest_powerstate tests enabled, but revert changes from: 66743f3 * leave VM poweroff tests enabled, but revert changes from: 87d6bda * bumped 'vcenter-test-container' version to '1.3.0' * updated test task names based on PR feedback
SUMMARY
updates 'vcenter-test-container' to '1.3.0' which has the latest tagged version of 'govmomi' (vcsim) 'v0.18.0'.
ISSUE TYPE
COMPONENT NAME
ANSIBLE VERSION
ADDITIONAL INFORMATION
This required updating a couple of the vmware_guest* module tests (needed to revert the
False
back toTrue
from ansible:87d6bdaf & ansible:66743f33)