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
kubectl apply: Deprecate --prune-whitelist in favor of --prune-allowlist #113116
Conversation
@brianpursley: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brianpursley 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 |
/hold |
Questions:
|
2a5b2cb
to
ba93dc2
Compare
ba93dc2
to
98bde95
Compare
98bde95
to
81d7db8
Compare
Thanks for adding all those tests!
Looking at that test, I think it is completely redundant with the test above it. Digging through the history, it looks like it was introduced as a regression test in #85613, but it definitely does not cover that case either. It seems to me it would need at least two changes to cover that PR: it would need to apply non-namespaced objects exclusively and set expectations on /lgtm |
@brianpursley I see this isn't auto-merging because you put a hold on it for review. Any further opinions you're looking for, or shall we remove that? |
@KnVerey I put the hold so it would wait for a review and not auto-merge /unhold |
Changes in kubectl apply --prune to support k8s Inclusive Naming Initiative: * Deprecated the --prune-whitelist flag. * Deprecated the PruneWhitelist field on ApplyFlags struct. * Removed PruneWhitelist field (not used anywhere) from ApplyOptions struct. * Added --prune-allowlist flag. * Added PruneAllowlist field on ApplyFlags struct. * Added unit tests for prune with allowlist This commit also fixes a bug where the command would fail if you specified the sameGVK multiple times for --allow-whitelist. Now it only attempts to prune the unique set of allowed GVKs.
81d7db8
to
f7ebf4d
Compare
Rebased |
/lgtm |
/test pull-kubernetes-e2e-kind-ipv6 |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Changes in kubectl apply --prune to support k8s Inclusive Naming Initiative:
This commit also fixes a bug where the command would fail if you specified the same GVK multiple times for --allow-whitelist. Now it only attempts to prune the unique set of allowed GVKs.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: