Skip to content

feat(standalone): add controlPlane.standalone.advertiseAddress field#4019

Merged
rlmcpherson merged 1 commit into
loft-sh:mainfrom
rlmcpherson:randallmcpherson/engcp-733-standalone-advertise-address
Jun 16, 2026
Merged

feat(standalone): add controlPlane.standalone.advertiseAddress field#4019
rlmcpherson merged 1 commit into
loft-sh:mainfrom
rlmcpherson:randallmcpherson/engcp-733-standalone-advertise-address

Conversation

@rlmcpherson

Copy link
Copy Markdown
Contributor

/kind feature

What does this pull request do?

Adds a controlPlane.standalone.advertiseAddress config field that pins the IP the standalone control plane advertises to the kubernetes and konnectivity Services.

Today this is only controllable via the VCLUSTER_STANDALONE_IP_ADDRESS env var (set indirectly by install-standalone.sh --node-ip). On multi-homed hosts (cloud VMs with both a public and a private NIC), ChooseHostInterface() selects the default-route IP — typically the public address, which is not reachable from the pod network — so the kubernetes/konnectivity EndpointSlices end up with unreachable IPs. This exposes a discoverable, first-class config field so operators can pin the in-cluster-reachable address without systemd drop-ins.

The vcluster-pro standalone code consumes it (prefer the config field, then the env var, then default-route detection) in a follow-up PR.

Please provide a short message that should be published in the vcluster release notes

Added controlPlane.standalone.advertiseAddress to pin the IP the standalone control plane advertises to the kubernetes/konnectivity Services (useful on multi-homed hosts).

Refs ENGCP-733

@rlmcpherson rlmcpherson requested review from a team as code owners June 16, 2026 18:07

@lizardruss lizardruss 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.

lgtm

@rlmcpherson rlmcpherson merged commit e95cfa5 into loft-sh:main Jun 16, 2026
15 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