Skip to content

Commit 58dcb28

Browse files
authored
Merge pull request #1661 from gianlucam76/service-side
fix: enable ServerSideApply when ForceConflicts is set for Helm v4 compatibility
2 parents c165067 + ebe3ad3 commit 58dcb28

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

controllers/handlers_helm.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3568,6 +3568,7 @@ func getHelmInstallClient(ctx context.Context, requestedChart *configv1beta1.Hel
35683568
installClient.Description = getDescriptionValue(requestedChart.Options)
35693569
installClient.PassCredentialsAll = getPassCredentialsToAllValue(requestedChart.Options)
35703570
installClient.TakeOwnership = getTakeOwnershipHelmValue(requestedChart.Options, false)
3571+
installClient.ServerSideApply = true
35713572
installClient.ForceConflicts = true
35723573
if actionConfig.RegistryClient != nil {
35733574
installClient.SetRegistryClient(actionConfig.RegistryClient)
@@ -3631,6 +3632,7 @@ func getHelmUpgradeClient(requestedChart *configv1beta1.HelmChart, actionConfig
36313632
upgradeClient.CaFile = registryOptions.caPath
36323633
upgradeClient.PassCredentialsAll = getPassCredentialsToAllValue(requestedChart.Options)
36333634
upgradeClient.TakeOwnership = getTakeOwnershipHelmValue(requestedChart.Options, true)
3635+
upgradeClient.ServerSideApply = "true"
36343636
upgradeClient.ForceConflicts = true
36353637

36363638
if actionConfig.RegistryClient != nil {

0 commit comments

Comments
 (0)