[mikenairn]

Summary

Fixes controller startup failure caused by duplicate controller names when using controller-runtime v0.19+.

Problem

The Kubernetes version upgrade in PR #684 introduced controller-runtime v0.23, which includes stricter validation introduced in v0.19 that requires all controllers registered with the same manager to have unique names.

Both the AuthConfigReconciler and AuthConfigStatusUpdater reconcile the same resource type (AuthConfig). Without an explicit controller name, controller-runtime derives the name from the resource type, causing both to get the name "authconfig".

This causes Authorino to fail on startup with:

{
  "level": "error",
  "ts": "2026-09-17T19:08:07Z",
  "logger": "authorino",
  "msg": "failed to create controller",
  "controller": "authconfigstatusupdate",
  "error": "controller with name authconfig already exists. Controller names must be unique to avoid multiple controllers reporting the same metric. This validation can be disabled via the SkipNameValidation option"
}

Solution

Explicitly set the controller name to "authconfigstatus" using the .Named() builder method in AuthConfigStatusUpdater.SetupWithManager() to ensure uniqueness.

References

• controller-runtime v0.19 breaking change: kubernetes-sigs/controller-runtime#2902 - "Validate controller names are unique"
• controller-runtime v0.19.0 release notes: https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.19.0
• Issue introduced in: #684 - "chore: upgraded kubernetes version to 1.30.x"

Summary by CodeRabbit

• Chores
  • Assigned an explicit name to the authentication configuration status controller for clearer identification.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 975007d1-61c1-4e90-b5fe-a450834c2248

📥 Commits

Reviewing files that changed from the base of the PR and between 0966088 and ec8874d.

📒 Files selected for processing (1)

• controllers/auth_config_status_updater.go

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

---

📝 Walkthrough

Walkthrough

SetupWithManager now assigns the controller the explicit name authconfigstatus.

Changes

Controller Naming

Layer / File(s)	Summary
Assign explicit controller name controllers/auth_config_status_updater.go	SetupWithManager passes authconfigstatus when constructing the controller.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to ec887

The controller naming change addresses the startup collision without introducing an identified runtime or integration risk.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarises the main change: assigning a unique controller name to the AuthConfig status updater. It is concise and specific.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the controller name
authconfigstatus now leads the game
SetupWithManager sets it clear
One small change brings order here
Hop by hop, the setup is done}

_{Comment @coderabbitai help to get the list of available commands.}