-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
openstack: Security groups update #1338
openstack: Security groups update #1338
Conversation
757f027
to
cbd42bd
Compare
data/data/openstack/main.tf
Outdated
@@ -28,7 +32,7 @@ module "service" { | |||
|
|||
swift_container = "${openstack_objectstorage_container_v1.container.name}" | |||
cluster_id = "${var.cluster_id}" | |||
cluster_domain = "${var.cluster_domain}" |
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.
cbd42bd
to
4be3041
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flaper87, trown 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 |
openstack: Security groups update
Through c8b3b55 (Merge pull request openshift#1338 from flaper87/sec-groups-update, 2019-05-01). I've left 8d0847c (openstack: Support setting network UUID via terraform variable, 2018-12-05, openshift#794) undocumented, since it seems like an unstable-enough user-facing API approach that I don't think we want to noise it about and deal with the fall-out when we change the API ;). That commit also made it into this history via 44a9cd3 (openshift#1294). https://github.com/openshift/cluster-version-operator/blob/master/Dockerfile#L8
Through c8b3b55 (Merge pull request openshift#1338 from flaper87/sec-groups-update, 2019-05-01). I've left 8d0847c (openstack: Support setting network UUID via terraform variable, 2018-12-05, openshift#794) undocumented, since it seems like an unstable-enough user-facing API approach that I don't think we want to noise it about and deal with the fall-out when we change the API ;). That commit also made it into this history via 44a9cd3 (openshift#1294). I've also left off eecf496 (openstack: remove neutron dns, 2019-02-19, openshift#1294), because I have no idea what that's about ;). I'll fill in an entry for it later once one of the OpenStack devs explains it to me :p.
we've changed the way security groups are filtered in the actuator. This patch updates the machine definition.