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
Add zone field to vsphere test cloudconfig #112427
Add zone field to vsphere test cloudconfig #112427
Conversation
/retest |
/assign @divyenpatel |
Please add a release note. |
/kind bug |
e2e in-tree test does not use this label? How does this e2e test config help? |
It is unfortunate that there is a difference between cloudconfig used by e2e vs actual cloudprovider. Ideally, they should have been the same because it is much easier to extract cloud-config and run tests with same configuration. So that is what this PR is trying to fix. It does not matter if tests use label field, but the tests should still be able to parse and run even if label field was there in actual cloudconfig. |
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.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: divyenpatel, gnufied The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/triage accepted |
…427-upstream-release-1.25 Automated cherry pick of #112427: Add zone field to vsphere test cloudconfig
…427-upstream-release-1.24 Automated cherry pick of #112427: Add zone field to vsphere test cloudconfig
This allows same configuration of cloudprovider and test.
Helps with e2e of vsphere intree driver