-
Notifications
You must be signed in to change notification settings - Fork 10
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
Refactor integration tests to allow execution in real vCenter #29
Refactor integration tests to allow execution in real vCenter #29
Conversation
885b50a
to
d159c2f
Compare
OT but do you think there's a chance to use this "real vCenter instance" CI for community.vmware and vmware.vmware_rest, too? |
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.
LGTM
@shellymiron please review it as well
6fea6d2
to
eaffa98
Compare
Hey Mario :) we will have a discussion about it tomorrow and we will keep this PR updated with what was decided |
5b3c873
to
8470734
Compare
@mariolenz hey! |
@shellymiron Probably not. But integration tests for community.vmware are still run with vCenter and ESXi 7.0 U3. It's really about time to start running those with a more current version. Latest release is 8.0 U3 and I think 7.0 will be EOL next(?) year. That said, I'm not unhappy with Zuul. It mostly works. But I understood that you don't want to work on those integration tests because of lacking the time / knowledge / experience / whatever to do this. I think it's both critical and necessary to run integration tests with a more current vSphere version, preferably 8.0 U3. And instead of trying to find someone who can update the Zuul integration tests accordingly or help me with yet another solution, I thought it would be the best thing if I could just re-use your solution. |
@mariolenz I see, so the main issue is with the vsphere version |
LGTM |
1b77b92
to
0025b12
Compare
8d137fb
to
3a88e69
Compare
@@ -0,0 +1,2 @@ | |||
vcenter_port: 443 |
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.
vcenter_port: 1080
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.
the variables in defaults/main.yml should be used for eco vcenter and they are overwritten by variable in vars.yml when the role is imported from run.yml playbook.
It looks like in the execution the port it's using is indeed 1080 but it fails:
TASK [vmware_appliance_info : Gather appliance info] ***************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Failed to connect to vCenter or ESXi API at 127.0.0.1:1080 : 404 Client Error: Not Found for url: https://127.0.0.1:1080/api"}
3a88e69
to
6744d98
Compare
6744d98
to
049dcd3
Compare
784a6e1
to
5fc50f6
Compare
SUMMARY
vmware_
prefixISSUE TYPE
ADDITIONAL INFORMATION
See ACA-1511