Skip to content

Commit

Permalink
Merge pull request #240 from openshift-cherrypick-robot/cherry-pick-2…
Browse files Browse the repository at this point in the history
…37-to-release-4.12

[release-4.12] OCPBUGS-12805: Make nested dual stack VIP configs respect EnableUnicast
  • Loading branch information
openshift-merge-robot committed May 11, 2023
2 parents 7552709 + cac8095 commit 128a1c4
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions pkg/monitor/dynkeepalived.go
Expand Up @@ -408,6 +408,13 @@ func KeepalivedWatch(kubeconfigPath, clusterConfigPath, templatePath, cfgPath st
}).Debug("EnableUnicast != enableUnicast from cfg file, update EnableUnicast value")
newConfig.EnableUnicast = curEnableUnicast
}
// Make sure the nested configs respect the current setting
// for EnableUnicast too. In EUS upgrades nodes may make it
// to a release with dual stack VIPs without having migrated
// to unicast first.
for i, _ := range *newConfig.Configs {
(*newConfig.Configs)[i].EnableUnicast = newConfig.EnableUnicast
}
updateUnicastConfig(kubeconfigPath, &newConfig)
curConfig = &newConfig
if doesConfigChanged(curConfig, appliedConfig) {
Expand Down

0 comments on commit 128a1c4

Please sign in to comment.