chore: Remove VPC/VPCAttachment CRDs, operator, and related code#87
Merged
Conversation
VPC and VPCAttachment CRD management is moving to a separate operator project. Galactic now focuses solely on the SRv6 data plane: the DaemonSet agent and CNI plugin receive pre-populated identifiers via CNI config and act on them directly. Removed: - pkg/apis/v1alpha/ — VPC and VPCAttachment CRD types - internal/operator/ — all controllers, webhook, cniconfig, identifier - internal/cmd/operator/ — operator subcommand - config/crd/bases/ — VPC and VPCAttachment CRD YAMLs - config/rbac/ — VPC/VPCAttachment role files - config/samples/ — VPC, VPCAttachment, and pod samples - config/webhook/ — pod mutation webhook manifests - config/certmanager/certificate-webhook.yaml - config/default/manager_webhook_patch.yaml - config/network-policy/allow-webhook-traffic.yaml Updated: - cmd/galactic/main.go — operator subcommand removed; agent/cni/version remain - Makefile — removed manifests, generate, run-operator, setup-envtest targets - go.mod/go.sum — dropped controller-runtime and ~60 transitive dependencies - config/ kustomization files — removed dangling references - CLAUDE.md, AGENTS.md, README.md, CONVENTIONS.md — reflect new scope Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
scotwells
previously approved these changes
May 22, 2026
- Remove test-operator job (envtest + controller tests are gone) - Remove make manifests generate steps from build and release jobs - Update test-unit to only run pkg/common/util tests - Remove test-operator from test-e2e needs - Remove VPC/VPCAttachment CRD verification and sample apply steps from e2e - Remove make manifests generate from release workflow Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove config/manager/ (deployment ran /galactic operator which no longer exists) - Remove config/default/metrics_service.yaml and manager_metrics_patch.yaml - Update config/default/kustomization.yaml to only include crd and rbac - Remove make deploy, undeploy, build-installer targets from Makefile - Remove make build-installer step and dist/install.yaml from release workflow - Remove make deploy and controller-manager wait from e2e job - Remove ginkgolinter from golangci config (no Ginkgo tests remain) - Remove dead api/* lll exclusion rule from golangci config Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
scotwells
approved these changes
May 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
VPC and VPCAttachment CRD management is moving to a separate operator project. Galactic now focuses solely on the SRv6 data plane: the DaemonSet agent and CNI plugin receive pre-populated identifiers via CNI config and act on them directly.
Removed:
Updated: