Closed dulek closed 6 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dulek Once this PR has been reviewed and has the lgtm label, please assign vincepri for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
Name | Link |
---|---|
Latest commit | 17fbc47392ae9fbf84de20a7d66d7a54e0c15bc8 |
Latest deploy log | https://app.netlify.com/sites/kubernetes-sigs-cluster-api-openstack/deploys/65fdb30098fbf000082bd7d8 |
Deploy Preview | https://deploy-preview-1968--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.
/retest
That's not my fault.
/test pull-cluster-api-provider-openstack-e2e-full-test /lgtm
let's see output of e2e-full job
PR needs rebase.
The experience of this PR resulted in the Filter->Param series, which is now merged.
What this PR does / why we need it: This makes options of SecurityGroupFilter pointers and adds validation that if ID is set, other options cannot be set.
Due to CEL rule cost budgeting I couldn't add rules validating tags in this way.
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 #Special notes for your reviewer:
TODOs:
/hold