Skip to content

kubectl-ate: add delete egress-policy - #1808

Merged
Bowei Du (bowei) merged 1 commit into
agent-substrate:mainfrom
ygao-g:kubectl-ate-egress-delete
Sep 30, 2026
Merged

Bowei Du (bowei) merged 1 commit into
agent-substrate:mainfrom
ygao-g:kubectl-ate-egress-delete

Conversation

@ygao-g

@ygao-g Yuan Gao (ygao-g) commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Part of #1550.

Adds kubectl ate delete egress-policy <actor-name> -a <atespace>

The command removes the actor's policy. It exits 1 when the actor does not exist
or has no policy. Also add optional --uid and --version flags, when specified,
the server refuses the delete when either no longer matches.

Testing

Tested on a kind cluster with the egress demo
and an API server built from this branch, using a CLI built from this branch:

  • guarded delete with a wrong version, then a wrong uid, then the right version
  • stale version after an update bumps the policy
  • plain delete on an actor without a policy, and on an actor that does not exist

🤖 This PR was developed with AI assistance. I have reviewed and tested all changes.

Comment thread cmd/kubectl-ate/README.md Resolved
@ygao-g Yuan Gao (ygao-g) changed the title kubectl-ate: add delete egress-policy [WIP] kubectl-ate: add delete egress-policy Sep 24, 2026
@ygao-g
Yuan Gao (ygao-g) marked this pull request as draft September 24, 2026 17:00
@ygao-g Yuan Gao (ygao-g) changed the title [WIP] kubectl-ate: add delete egress-policy kubectl-ate: add delete egress-policy Sep 24, 2026
@ygao-g
Yuan Gao (ygao-g) marked this pull request as ready for review September 24, 2026 18:41
@bowei
Bowei Du (bowei) added this pull request to the merge queue Sep 29, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 29, 2026
@ygao-g

Copy link
Copy Markdown
Collaborator Author

Bowei Du (@bowei) PTAL as github dismisses previous approvals after rebase

@ygao-g
Yuan Gao (ygao-g) marked this pull request as draft September 29, 2026 21:19
@ygao-g
Yuan Gao (ygao-g) force-pushed the kubectl-ate-egress-delete branch 2 times, most recently from c5a81bc to e506267 Compare September 29, 2026 22:09
@ygao-g
Yuan Gao (ygao-g) force-pushed the kubectl-ate-egress-delete branch 3 times, most recently from 3628de2 to 48ee436 Compare September 29, 2026 22:17
"delete egress-policy <actor> -a <atespace>" removes the actor's egress
policy. Fails if the actor is missing, or the actor has no policy.

Optional --uid and --version flags guard the delete: the server refuses
it when either no longer matches the stored policy.
@ygao-g
Yuan Gao (ygao-g) marked this pull request as ready for review September 29, 2026 23:39
@bowei
Bowei Du (bowei) added this pull request to the merge queue Sep 30, 2026
Merged via the queue into agent-substrate:main with commit 1282773 Sep 30, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli area/network kind/feature An enhancement / feature request or implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants