Skip to content

[Backport release-1.6] fix(kubeovn-webhook): restrict the webhook port to the API server - #3975

Merged
Andrei Kvapil (kvaps) merged 2 commits into
release-1.6from
backport-3955-to-release-1.6
Aug 27, 2026
Merged

Andrei Kvapil (kvaps) merged 2 commits into
release-1.6from
backport-3955-to-release-1.6

Conversation

@github-actions

Copy link
Copy Markdown

Description

Backport of #3955 to release-1.6.

@github-actions

Copy link
Copy Markdown
Author

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-3955-to-release-1.6
git worktree add --checkout .worktree/backport-3955-to-release-1.6 backport-3955-to-release-1.6
cd .worktree/backport-3955-to-release-1.6
git reset --hard HEAD^
git cherry-pick -x cdc34d6e344585050643248cf553f0f3635522be 6663c44dd52fad28c80b0f1d1feb4234a11aef63 a7cef35d88ac1c410cb63b70f7279e0fe132e765
git push --force-with-lease

Backport of #3955. The automated cherry-pick carried only the first
commit of that PR and committed the values.yaml conflict, so this
resolution brings the branch to what actually merged to main: the port
comes from a single helper feeding the container port, the Service
targetPort and the policy, the rationale sits in template comments
rather than being rendered into every manifest, and the dead egress
default-deny toggle is gone.

Two deliberate differences from main. The image stays on the digest this
release line already ships rather than main's older one, and the
--client-ca-file argument is left out along with the values key it
reads: both belong to #2698, which was never backported here, and the
image on this line does not take the flag.

Assisted-By: Claude
Signed-off-by: Andrei Kvapil <[email protected]>
@kvaps
Andrei Kvapil (kvaps) marked this pull request as ready for review August 27, 2026 16:15
@kvaps
Andrei Kvapil (kvaps) merged commit 176c410 into release-1.6 Aug 27, 2026
7 of 8 checks passed
@kvaps
Andrei Kvapil (kvaps) deleted the backport-3955-to-release-1.6 branch August 27, 2026 16:16
@github-actions github-actions Bot added size/L This PR changes 100-499 lines, ignoring generated files area/networking Issues or PRs related to networking (ingress, gateway, vpn, metallb, kube-ovn) area/release Issues or PRs related to release tooling (changelog, backport, release pipeline) kind/bug Categorizes issue or PR as related to a bug labels Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking Issues or PRs related to networking (ingress, gateway, vpn, metallb, kube-ovn) area/release Issues or PRs related to release tooling (changelog, backport, release pipeline) kind/bug Categorizes issue or PR as related to a bug size/L This PR changes 100-499 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant