Skip to content

Commit 3b5becb

Browse files
committed
fix injection of extra arg
was injected to commands (as previously) but should now be injected to args with latest gardener changes
1 parent a9c9463 commit 3b5becb

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pkg/webhook/controlplane/ensurer.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ func (e *ensurer) EnsureMachineControllerManagerDeployment(_ context.Context, _
6565
}
6666

6767
sidecarContainer := machinecontrollermanager.ProviderSidecarContainer(newObj.Namespace, azure.Name, image.String())
68-
sidecarContainer.Command = append(sidecarContainer.Command, "--machine-pv-reattach-timeout=150s")
68+
sidecarContainer.Args = append(sidecarContainer.Args, "--machine-pv-reattach-timeout=150s")
6969

7070
newObj.Spec.Template.Spec.Containers = extensionswebhook.EnsureContainerWithName(newObj.Spec.Template.Spec.Containers, sidecarContainer)
7171
return nil

pkg/webhook/controlplane/ensurer_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -514,7 +514,7 @@ var _ = Describe("Ensurer", func() {
514514
Expect(deployment.Spec.Template.Spec.Containers).To(BeEmpty())
515515
Expect(ensurer.EnsureMachineControllerManagerDeployment(context.TODO(), nil, deployment, nil)).To(BeNil())
516516
expectedContainer := machinecontrollermanager.ProviderSidecarContainer(deployment.Namespace, "provider-azure", "foo:bar")
517-
expectedContainer.Command = append(expectedContainer.Command, "--machine-pv-reattach-timeout=150s")
517+
expectedContainer.Args = append(expectedContainer.Args, "--machine-pv-reattach-timeout=150s")
518518
Expect(deployment.Spec.Template.Spec.Containers).To(ConsistOf(expectedContainer))
519519
})
520520
})

0 commit comments

Comments
 (0)