-
Notifications
You must be signed in to change notification settings - Fork 105
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
[os_net_setup] Add --availability-zone-hint to network creation #2497
[os_net_setup] Add --availability-zone-hint to network creation #2497
Conversation
Thanks for the PR! ❤️ |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pablintino 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 |
(non-blocking) suggestion: I see some vars used in the template that are missing of some context and format. WDYT about adding that to the default file: https://github.com/openstack-k8s-operators/ci-framework/blob/main/roles/os_net_setup/defaults/main.yml to see an example? I'm thinking if it might make sense to rework eventually in the README file to briefly explain each of the possibilities of: cifmw_os_net_setup_config Apart of this: lgtm |
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.
I agree with @evallesp
Please add availability_zone_hints to the role defaults, others are missing but they can be added in a follow up if we get time to refactor this role.
This PR adds the option to configure external networks for certain AZs. In order to do this, the list `availability_zone_hints` should be configured as port of `cifmw_os_net_setup_config`.
9766082
to
dc3d0cf
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/de855c24e2634b82be544cbafc270baa ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 57m 02s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1a95eda3d3d44e2b8d13236505d0df66 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 49m 08s |
recheck |
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
Thanks @eduolivares ++
1fcdd76
into
openstack-k8s-operators:main
This PR adds the option to configure external networks for certain AZs. In order to do this, the list
availability_zone_hints
should be configured as port ofcifmw_os_net_setup_config
.