Skip to content

Move non-cluster-host typha into the enterprise extension - #5189

Merged
caseydavenport merged 2 commits into
tigera:masterfrom
caseydavenport:casey-devariant-noncluster-host
Aug 18, 2026
Merged

Move non-cluster-host typha into the enterprise extension#5189
caseydavenport merged 2 commits into
tigera:masterfrom
caseydavenport:casey-devariant-noncluster-host

Conversation

@caseydavenport

Copy link
Copy Markdown
Member

Description

Non-cluster-host typha now renders from the enterprise installation extension instead of core. What leaves core:

  • the second typha deployment and service
  • non-cluster-host fields on the typha/node TLS struct
  • HostEndpoint informer and the second autoscaler
  • reading the NonClusterHost resource, and parsing the BYO node cert

The policy for the non-cluster-host typha still renders alongside the other component policies, same as before.

The typha autoscaler moved to its own package so the enterprise extension can reach it without an import cycle, and it takes a controller-runtime client now instead of a rest.Config plus a clientset.

One behavior change worth calling out: the serving keypair for the non-cluster-host typha is only created when a NonClusterHost resource exists. Before it was created on every reconcile, in both variants.

None

The manager cache already serves nodes and host endpoints, so the autoscaler needs
neither its own informers nor a clientset. A replica counter function replaces the
noncluster-host flag.
The typha deployment, service, policy, and autoscaler for non-cluster hosts
now render from pkg/enterprise/installation.
@caseydavenport
caseydavenport force-pushed the casey-devariant-noncluster-host branch from 0b10d75 to 0311bf6 Compare August 18, 2026 16:42
@caseydavenport
caseydavenport merged commit 1d2d110 into tigera:master Aug 18, 2026
6 checks passed
@caseydavenport
caseydavenport deleted the casey-devariant-noncluster-host branch August 18, 2026 21:19
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.

4 participants