Skip to content

Move the log collector and intrusion detection controllers into pkg/enterprise - #5276

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

Move the log collector and intrusion detection controllers into pkg/enterprise#5276
caseydavenport wants to merge 4 commits into
tigera:masterfrom
caseydavenport:casey-enterprise-controllers-3

Conversation

@caseydavenport

@caseydavenport caseydavenport commented Aug 29, 2026

Copy link
Copy Markdown
Member

Description

Third batch for CORE-13484. The log collector and intrusion detection controllers move under the Enterprise tree, together with the OpenTelemetry collector and intrusion detection render packages that nothing in core imports once they go.

  • Intrusion detection reached into the installation controller for the Typha and node TLS config, which would have made the Enterprise tree import core's installation controller and cycle through its test suite. Those helpers move to the shared controller utilities instead, along with the operator-namespace ConfigMap read they depend on.
  • The OpenTelemetry collector's identity constants stay in core, since the monitoring render builds its ServiceMonitor and cannot import the package that renders it.

Stacked on #5274. 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.
…nterprise

The Typha and node TLS helpers move to the shared controller utilities, which
is what lets the intrusion detection controller stop importing the installation
controller.
@caseydavenport
caseydavenport force-pushed the casey-enterprise-controllers-3 branch from a964b30 to 8d08fbf 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