Skip to content

fix: allocate Envoy Gateway/Grafana NodePorts dynamically - #35

Draft
scotwells wants to merge 1 commit into
mainfrom
fix/remove-hardcoded-nodeports
Draft

fix: allocate Envoy Gateway/Grafana NodePorts dynamically#35
scotwells wants to merge 1 commit into
mainfrom
fix/remove-hardcoded-nodeports

Conversation

@scotwells

Copy link
Copy Markdown
Contributor

Summary

  • Fixed nodePort/hostPort values (30443 HTTPS, 30080 HTTP, 30000 Grafana) prevented running more than one test-infra kind cluster on the same Docker host — Docker refuses to bind the same host port twice, so cluster-up with a second CLUSTER_NAME would fail on port conflicts.
  • Removes extraPortMappings from cluster/kind-config.yaml and the nodePort overrides from the EnvoyProxy/Grafana Service patches. Kubernetes now allocates an ephemeral NodePort per cluster instead.
  • Adds task gateway-port-forward / task grafana-port-forward as the recommended way to reach these services now that the port isn't fixed, and updates docs/messaging accordingly.

Motivation

Needed for dns-operator's new local test environment, which stands up two concurrent kind clusters (control + edge) on the same host to exercise PowerDNS Lightningstream edge federation end-to-end.

Test plan

  • kustomize build components/envoy-gateway-operator/gateway-resources builds cleanly with no nodePort field
  • task cluster-up CLUSTER_NAME=a and task cluster-up CLUSTER_NAME=b succeed concurrently on the same host
  • task gateway-port-forward / task grafana-port-forward reach the expected services

Fixed nodePort/hostPort values (30443, 30080, 30000) prevented running
more than one test-infra cluster on the same Docker host, since kind's
extraPortMappings bind to fixed host ports and Docker refuses to bind
the same host port twice.

Drop the extraPortMappings from cluster/kind-config.yaml and the
nodePort overrides from the EnvoyProxy/Grafana Service patches so
Kubernetes assigns an ephemeral NodePort per cluster instead. Add
gateway-port-forward/grafana-port-forward tasks as the recommended way
to reach these services now that the port isn't fixed.
@scotwells
scotwells marked this pull request as draft July 23, 2026 02:31
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.

1 participant