chore: bump cosmos dependency and clean up go.mod#123
Merged
Conversation
- Update go.miloapis.com/cosmos to latest commit - Move k8s.io/api from indirect to direct dependency - Remove unused spf13/cobra and related dependencies
- Update CNI to use new BGPRouter/BGPPeer/BGPAdvertisement CRDs - Rename deploy/galactic-agent → deploy/galactic-router - Update Dockerfile to build galactic-router instead of galactic-agent - Update devcontainer, CI scripts, Taskfile for new layout - Update documentation to reflect router architecture
ecv
previously approved these changes
Jun 23, 2026
The cosmos dependency bump changed package paths; BGPProvider/BGPInstance are replaced by BGPRouter in the new API. Remove the agent, bootstrap, and gobgp packages that relied on the old paths, update the CNI to use the new BGPRouter API, and drop the now-unused gobgp/grpc direct dependencies. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
# Conflicts: # Taskfile.yaml # internal/cni/cni.go # internal/cni/cni_test.go
chore: update CNI and infrastructure for galactic-router
The galactic-router build targets in Taskfile and Dockerfile referenced cmd/galactic-router/main.go, which does not exist on this branch (the router implementation and its internal packages land in a later stacked PR). go build treated the missing file path as an import path, failing with "package cmd/galactic-router/main.go is not in std" and breaking the Build CI job. Drop the premature router build/copy steps so each binary that actually has source on this branch builds cleanly. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
cosmos is now a regular module dependency (this PR's go.mod cleanup), no
longer a local ../cosmos replace. The Dockerfile still copied it from a
"cosmos" named build context and scripts/ci.sh passed
--build-context cosmos=../cosmos, which does not exist in CI, breaking the
E2E image build ("failed to get build context cosmos: stat ../cosmos").
Remove both; go mod download now fetches cosmos from the module proxy.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ecv
approved these changes
Jun 23, 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.
Summary
Bump cosmos dependency and clean up go.mod dependencies.
Changes
This is a prerequisite for all subsequent PRs in the galactic-router rewrite.