Skip to content

Move four more Enterprise-only controllers into pkg/enterprise - #5274

Open
caseydavenport wants to merge 4 commits into
tigera:masterfrom
caseydavenport:casey-enterprise-controllers-batch
Open

Move four more Enterprise-only controllers into pkg/enterprise#5274
caseydavenport wants to merge 4 commits into
tigera:masterfrom
caseydavenport:casey-enterprise-controllers-batch

Conversation

@caseydavenport

@caseydavenport caseydavenport commented Aug 28, 2026

Copy link
Copy Markdown
Member

Description

Second batch for CORE-13484. Four more Enterprise-only controllers move under the Enterprise tree, along with the render packages nothing in core imports.

  • Application layer, egress gateway, packet capture and OpenTelemetry collector controllers, each contributed through the registration list rather than named by the core manager setup.
  • Their variant checks go with them, since registration is what gates a contributed controller.
  • Render code for the application layer and egress gateway moves too. The OpenTelemetry collector's stays behind until the log collector controller moves, and packet capture renders out of the top-level render package, which is its own step.

Stacked on #5273, which carries the registration seam. Review the last commit.

Related: CORE-13484

Release Note

None

A variant contributes its own controllers through the controller list on the
controller options, which the manager setup iterates instead of naming them.
The Enterprise tree mirrors the core tree it came from, keeping controllers
apart from the extension packages that share the directory.
Application layer, egress gateway, packet capture and OpenTelemetry collector,
with the application layer and egress gateway render packages.
@caseydavenport
caseydavenport force-pushed the casey-enterprise-controllers-batch branch from 0a4f416 to e9cba89 Compare August 29, 2026 15:36
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.

3 participants