Closed EmilienM closed 6 months ago
Name | Link |
---|---|
Latest commit | 350b7e60ca116e8349035a65a263d4cddab651eb |
Latest deploy log | https://app.netlify.com/sites/kubernetes-sigs-cluster-api-openstack/deploys/65d620b09687bc00083160ff |
Deploy Preview | https://deploy-preview-1892--kubernetes-sigs-cluster-api-openstack.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
We need https://github.com/kubernetes-sigs/cluster-api-provider-openstack/pull/1826 to merge first.
/lgtm
/lgtm
/hold cancel
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: EmilienM, mdbooth
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
What this PR does / why we need it:
Move
OpenStackCluster.Spec.AllowAllInClusterTraffic
underOpenStackCluster.Spec.ManagedSecurityGroups
.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #1878TODOs: