Skip to content

Commit

Permalink
UPSTREAM: <drop>: Switch default load balancer health check to shared
Browse files Browse the repository at this point in the history
This fix should go in to the CCCMO, but carry this commit to enforce the
default until OCPBUGS-24486 is addressed there.

Signed-off-by: Nolan Brubaker <nolan@nbrubaker.com>
  • Loading branch information
nrb committed Jan 18, 2024
1 parent 22cdf9f commit 45890ca
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pkg/provider/azure.go
Original file line number Diff line number Diff line change
Expand Up @@ -588,8 +588,10 @@ func (az *Cloud) InitializeCloudFromConfig(ctx context.Context, config *Config,
}
}

// This carried change should be dropped when OCPBUGS-24486 is merged into the CCM to set the shared mode.
// Changing it here to ensure there's no gap in functionality that breaks paylod.
if config.ClusterServiceLoadBalancerHealthProbeMode == "" {
config.ClusterServiceLoadBalancerHealthProbeMode = consts.ClusterServiceLoadBalancerHealthProbeModeServiceNodePort
config.ClusterServiceLoadBalancerHealthProbeMode = consts.ClusterServiceLoadBalancerHealthProbeModeShared
} else {
supportedClusterServiceLoadBalancerHealthProbeModes := sets.New(
strings.ToLower(consts.ClusterServiceLoadBalancerHealthProbeModeServiceNodePort),
Expand Down

0 comments on commit 45890ca

Please sign in to comment.