Skip to content

Move Galactic VPC reconciliation to NSO and retain BGP status - #886

Merged
scotwells merged 2 commits into
mainfrom
codex/galactic-attachment-reprogram
Oct 16, 2026
Merged

scotwells merged 2 commits into
mainfrom
codex/galactic-attachment-reprogram

Conversation

@scotwells

@scotwells scotwells commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Move VPC creation, subnet programming status, VPCAttachment and NAD reconciliation from Galactic's cell controller to NSO's cell manager. Galactic's cell role now owns BGP advertisement status projection and traffic policy.
  • Preserve the Galactic BGP handoff: an attachment group change invalidates stale NSO interface and claim programming, and the new-group advertisement restores it.
  • Remove the Galactic attachment mode flag and VPC/NAD write permissions; update the migration runbook.

Verification

  • go test ./internal/cloud/controller ./internal/cloud/galactic ./internal/cloud/webhook ./cmd/galactic-controller -count=1 passed.
  • go vet ./internal/cloud/controller ./internal/cloud/webhook ./cmd/galactic-controller passed.
  • task ci remains blocked on this macOS host because llvm-strip is unavailable; GitHub CI covers the full pipeline.

Rollout

NSO's Galactic reconcilers are disabled by default. Install CRDs, stop the old VPC cell writer, and enable NSO before unsuspending Galactic's BGP status controller. Staging infra remains a draft migration and is not ready to merge.

Companion PRs: datum-cloud/network-services-operator#662 and datum-cloud/infra#7107.

@scotwells
scotwells requested a review from a team as a code owner October 16, 2026 14:51
@scotwells scotwells changed the title Reprogram NSO claims after Galactic API group migration Move Galactic VPC reconciliation to NSO and retain BGP status Oct 16, 2026
@scotwells
scotwells merged commit 6a6af5d into main Oct 16, 2026
16 checks passed
@scotwells
scotwells deleted the codex/galactic-attachment-reprogram branch October 16, 2026 15:39
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