-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Follow on for #5744 #5862
Follow on for #5744 #5862
Conversation
If we skip it, we can't test it. We do expect that most users will use a lifecycle that only warns though.
Fixes the case where we mix use of specified & default SGs.
e.g. each master SGs need to be configured to talk to each master SG
This ensures we are consistently naming our rules
Also add comments about why we don't set e.g. RemoveExtraRules
b02608f
to
81cadec
Compare
cc @rdrgmnzs |
/test pull-kops-e2e-kubernetes-aws |
Thanks for the cleanup @justinsb /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb, rdrgmnzs 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 |
/test pull-kops-e2e-kubernetes-aws |
Address some cases where users mix security groups in a role, or mix
override & default security groups.