refactor: rename authproxy-routes ConfigMap to authbridge-routes for consistency
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 18
- Forks
- 50
- Avg merge
- 3d 3h
- Merged PRs (30d)
- 8
Description
Problem
The namespace-level default routes ConfigMap is named authproxy-routes, while per-agent routes ConfigMaps use the pattern authbridge-routes-<crName>. This naming inconsistency is confusing:
// Current naming:
AuthproxyRoutesConfigMapName = "authproxy-routes" // namespace default
"authbridge-routes-" + crName // per-agent
The name authproxy is legacy terminology from before the component was renamed to AuthBridge.
Proposed Solution
Rename to use consistent authbridge- prefix:
AuthBridgeRoutesConfigMapName = "authbridge-routes" // namespace default
"authbridge-routes-" + crName // per-agent (unchanged)
Breaking Change Considerations
This is a breaking change for any namespace that has a pre-existing authproxy-routes ConfigMap. Migration options:
Option A: Hard break (v1.0 boundary)
- Rename in one PR
- Document in release notes
- Users must manually rename their ConfigMap
Option B: Deprecation period
- Support both names (check both ConfigMaps, prefer new name)
- Log deprecation warning when old name is found
- Remove support after 2-3 releases
Option C: Automatic migration
- Webhook checks for
authproxy-routes, copies toauthbridge-routes - Annotates old ConfigMap with deprecation notice
- More complex, but smoothest UX
Scope
Files to update:
internal/webhook/injector/namespace_config.go- constant definitioninternal/webhook/injector/volume_builder.go- volume name referencesinternal/webhook/injector/container_builder.go- volume mount name- Documentation:
docs/authbridge/demos.md,docs/authbridge-webhook.md - Test files
Related
- PR #517 added per-agent
authbridge-routes-<crName>ConfigMaps - The inconsistency was noticed during #517 review
Recommendation
Wait until v1.0 release boundary to make this breaking change. Use Option A (hard break) since the namespace-level authproxy-routes ConfigMap is rarely used in practice (most users rely on per-agent routes from AgentRuntime CRDs).
/cc @alantech
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read internal/webhook/injector/namespace_config.go, volume_builder.go, and container_builder.go, then inspect the related tests and the documentation files listed in the issue. First confirm whether the v1.0 boundary should use the hard break or another migration option; done means the selected naming and migration behavior are reflected consistently in code, tests, and docs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, kubernetes
- Domain
- infrastructure
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100