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 0bfe8d5 commit 532b643
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 532b643

Please sign in to comment.