Skip to content

[release-v1.42] Strip apiserver endpoint env vars on non-cluster-host Typha by default#4847

Merged
caseydavenport merged 1 commit into
tigera:release-v1.42from
caseydavenport:casey-nch-typha-strip-v1.42
May 22, 2026
Merged

[release-v1.42] Strip apiserver endpoint env vars on non-cluster-host Typha by default#4847
caseydavenport merged 1 commit into
tigera:release-v1.42from
caseydavenport:casey-nch-typha-strip-v1.42

Conversation

@caseydavenport
Copy link
Copy Markdown
Member

Cherry-pick of #4846 (follow-up to #4840 / #4842) to release-v1.42. Requested for inclusion in the v3.23.1 patch release.

The non-cluster-host Typha runs pod-networked but inherits KUBERNETES_SERVICE_HOST/PORT from the host-network endpoint. The earlier cherry-pick (#4842) only overrode those env vars when a pod-network endpoint was explicitly configured, which left the MKE failure mode in place when the customer hadn't set one. This change always strips the inherited host-network values so kubelet's default-service injection takes over, then re-adds the pod-network endpoint if one was configured.

Clean cherry-pick.

Related: https://tigera.atlassian.net/browse/CI-1987

NONE

tigera#4846)

* Strip apiserver endpoint env vars on non-cluster-host Typha by default

The non-cluster-host Typha is pod-networked, so it should use the
default in-cluster kubernetes Service that kubelet injects into every
pod. Always strip the inherited KUBERNETES_SERVICE_HOST/PORT (which
come from the host-network endpoint, e.g. proxy.local on MKE) and only
override when a pod-network endpoint is explicitly configured.

* Use slices.DeleteFunc instead of bespoke env-var filter helper
@caseydavenport caseydavenport requested a review from a team as a code owner May 22, 2026 00:04
@marvin-tigera marvin-tigera added this to the v1.42.1 milestone May 22, 2026
@caseydavenport caseydavenport merged commit 5fd2db8 into tigera:release-v1.42 May 22, 2026
4 checks passed
@caseydavenport caseydavenport deleted the casey-nch-typha-strip-v1.42 branch May 22, 2026 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants