Skip to content

feat: support GALACTIC_GOBGP_GRPC_SERVER and GALACTIC_GOBGP_GRPC_PORT env vars#133

Merged
privateip merged 1 commit into
mainfrom
feat/support-gobgp-envvars
Jun 25, 2026
Merged

feat: support GALACTIC_GOBGP_GRPC_SERVER and GALACTIC_GOBGP_GRPC_PORT env vars#133
privateip merged 1 commit into
mainfrom
feat/support-gobgp-envvars

Conversation

@privateip

Copy link
Copy Markdown
Contributor

Summary

This patch adds two new env vars to Galatic intended assist with troubleshooting. By default, the GoBGP gRPC server is disabled when galactic-router starts up. This enables the gRPC server. When enabled, operators and developers can use the gobgp cli to introspect the GoBGP state.

Changes

  • Add GALACTIC_ENABLE_GOBGP_GRPC_SERVER env var to toggle the GoBGP gRPC API server on/off
  • Add GALACTIC_GOBGP_GRPC_PORT env var to configure the gRPC listen port (default 50051)
  • Wire grpcListenAddress through NewRuntimeFactory into the GoBGP server Config
  • Extract buildServerOptions from newBgpServer for testability
  • Add unit tests for parseGrpcPort, parseGrpcListenAddress, and buildServerOptions

… env vars

- Add GALACTIC_ENABLE_GOBGP_GRPC_SERVER env var to toggle the GoBGP gRPC API server on/off
- Add GALACTIC_GOBGP_GRPC_PORT env var to configure the gRPC listen port (default 50051)
- Wire grpcListenAddress through NewRuntimeFactory into the GoBGP server Config
- Extract buildServerOptions from newBgpServer for testability
- Add unit tests for parseGrpcPort, parseGrpcListenAddress, and buildServerOptions
@privateip privateip requested a review from a team as a code owner June 25, 2026 21:53
@privateip privateip requested a review from JoseSzycho June 25, 2026 21:53

@scotwells scotwells left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where do we document all of the env variables that are supported? I normally like using CLI args so it's easier to see all config options via --help.

@privateip

Copy link
Copy Markdown
Contributor Author

Yeah i agree, i need to revise galactic-router and galactic-cni to accept command line option and/or env var. will tackle that shortly

@privateip privateip merged commit 7e72e2c into main Jun 25, 2026
5 checks passed
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