Skip to content

Move the operator's startup checks for Enterprise into pkg/enterprise - #5204

Merged
caseydavenport merged 2 commits into
tigera:masterfrom
caseydavenport:casey-startup-extension
Aug 18, 2026
Merged

Move the operator's startup checks for Enterprise into pkg/enterprise#5204
caseydavenport merged 2 commits into
tigera:masterfrom
caseydavenport:casey-startup-extension

Conversation

@caseydavenport

Copy link
Copy Markdown
Member

Description

Continues moving Enterprise-only logic behind the extensions boundary, this time the startup checks that run before any controller starts. Three of them move into the Enterprise package as plain functions that main calls:

  • verifying the Enterprise CRDs are installed
  • the Elasticsearch internal/external certificate check
  • which namespaces the operator refuses to install itself into

All three now have unit tests, where they previously had none.

Small behavior changes that fell out of it:

  • The Elasticsearch certificate check ran twice on cloud builds, from two near-duplicate copies of the same code. It runs once now.
  • Calico installs ran that check too, where it was always a no-op since the secrets only exist for Enterprise. Skipped there now.
  • Dropped "calico-apiserver" from the refused namespaces, since nothing uses that namespace anymore.

Release Note

None

The Enterprise CRD gate, the Elasticsearch certificate exclusivity check, and
the list of managed namespaces all lived in main.go, so a Calico build carried
them too. They move to pkg/enterprise, where the private main can call them
directly once that repo has its own binary.
@caseydavenport
caseydavenport force-pushed the casey-startup-extension branch from 8c0f1cb to 2bfab0d Compare August 18, 2026 16:30
@caseydavenport
caseydavenport merged commit 6112566 into tigera:master Aug 18, 2026
6 checks passed
@caseydavenport
caseydavenport deleted the casey-startup-extension branch August 18, 2026 20:05
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