fix: Add ENABLE_SERVICE_NETWORK_OVERRIDE env var to helm chart #857
+2
−0
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.
What type of PR is this?
bug
Which issue does this PR fix:
N/A - Bug discovered while testing the ServiceNetworkOverrideMode feature introduced in recent releases.
What does this PR do / Why do we need it:
Fixes a critical bug where the ServiceNetworkOverrideMode feature is non-functional because the helm chart is missing the environment variable mapping.
The controller code supports
ENABLE_SERVICE_NETWORK_OVERRIDE, andpkg/config/controller_config.goparses it from the environment, buthelm/templates/deployment.yamldoesn't include the mapping from helm value to environment variable. This makes the feature completely unusable via helm deployments.Impact: Any deployment using
enableServiceNetworkOverride: truein helm values will silently fail - the controller never receives the environment variable and override mode never activates.If an issue # is not available please add repro steps and logs from aws-gateway-controller showing the issue:
Reproduction Steps
helm install gateway-api-controller ... --set enableServiceNetworkOverride=trueLog Evidence
Controller logs show it's using Gateway name as network name (proof override mode isn't active):
{"level":"info","msg":"Creating service with service network association ... (networks: [my-gateway])"}Expected with override mode:
(networks: [sn-xxxxxxxxxxxxx])(the configured DEFAULT_SERVICE_NETWORK)Testing done on this change:
Tested in EKS cluster
enableServiceNetworkOverride = "true"helm valueAutomation added to e2e:
No - This is a helm chart configuration bug, not a functional feature addition. Existing ServiceNetworkOverride tests cover the feature itself.
Will this PR introduce any new dependencies?:
No - This only adds a missing helm template line.
Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this PR introduce any user-facing change?:
Yes - makes ServiceNetworkOverrideMode feature actually work via helm.
Do all end-to-end tests successfully pass when running
make e2e-test?:Not run for this PR - this is a 2-line helm template fix with no logic changes. The ServiceNetworkOverrideMode feature itself already has test coverage in
pkg/gateway/model_build_lattice_service_test.go.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.