Skip to content

Move the monitor controller into pkg/enterprise - #5273

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

Move the monitor controller into pkg/enterprise#5273
caseydavenport wants to merge 3 commits into
tigera:masterfrom
caseydavenport:casey-enterprise-controllers

Conversation

@caseydavenport

@caseydavenport caseydavenport commented Aug 28, 2026

Copy link
Copy Markdown
Member

Description

First of the controller relocations for CORE-13484. The monitoring controller is Enterprise-only, so it moves under the Enterprise package along with the registration seam the rest of the moves need.

  • A variant contributes its own controllers through a list on the controller options. The core manager setup iterates that list rather than naming each type, so it no longer knows the Enterprise controllers exist.
  • Registration is what gates a contributed controller, so the moved controller drops its own variant check.
  • The Enterprise tree mirrors the core tree it came from, so a controller lands in pkg/enterprise/controller and its render code will land in pkg/enterprise/render. Flat would collide with the extension packages already sharing that directory.
  • The render package stays where it is. Several other Enterprise controllers still depend on it, and it moves once they do.

The rest of the Enterprise-only controllers follow in separate PRs, one subsystem at a time.

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.
@caseydavenport
caseydavenport force-pushed the casey-enterprise-controllers branch from 914a82a to fca5347 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