Skip to content

fix: correct node-selector label domain to galactic.datumapis.com#225

Merged
privateip merged 1 commit into
mainfrom
fix/node-label-domain
Jul 10, 2026
Merged

fix: correct node-selector label domain to galactic.datumapis.com#225
privateip merged 1 commit into
mainfrom
fix/node-label-domain

Conversation

@privateip

Copy link
Copy Markdown
Contributor

Summary

  • The galactic.datum.net/node label used to opt nodes into the route-reflector (control) and per-node (edge) roles used the wrong domain
  • Renamed to galactic.datumapis.com/node across config/, all containerlab lab manifests (node configs, fabric/control daemonset patches), and docs (README.md, AGENTS.md/CLAUDE.md, docs/agents/ARCHITECTURE.md, deploy/containerlab/AGENTS.md)
  • Internal Go annotation keys under the same old domain (allocated-subnet, srv6-sid, netns, config-hash) were intentionally left unchanged — out of scope for this label fix

Test plan

  • kubectl kustomize config/ builds cleanly (verified locally)
  • Re-label containerlab nodes with galactic.datumapis.com/node before next lab bring-up

The galactic.datum.net/node label used to opt nodes into the
route-reflector and edge roles pointed at the wrong domain; rename it
to galactic.datumapis.com/node across config/, the containerlab lab
manifests, and docs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@privateip privateip requested a review from a team as a code owner July 10, 2026 19:41
@privateip privateip requested a review from mksinghtx July 10, 2026 19:41
@privateip privateip merged commit cc37dbb into main Jul 10, 2026
8 checks passed
@privateip privateip deleted the fix/node-label-domain branch July 10, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants