Skip to content

Commit e638fb3

Browse files
aaronfernmaboehm
andauthored
Fix panic if provider does not support Driver.InitializeMachine (#1032) (#1047)
Co-authored-by: Marcel Boehm <[email protected]>
1 parent 27dbe04 commit e638fb3

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

pkg/util/provider/machinecontroller/machine.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -654,7 +654,9 @@ func (c *controller) triggerCreationFlow(ctx context.Context, createMachineReque
654654
if c.targetCoreClient == nil {
655655
// persist addresses from the InitializeMachine and CreateMachine responses
656656
clone := clone.DeepCopy()
657-
addresses.Insert(initResponse.Addresses...)
657+
if initResponse != nil {
658+
addresses.Insert(initResponse.Addresses...)
659+
}
658660
clone.Status.Addresses = buildAddressStatus(addresses, nodeName)
659661
if _, err := c.controlMachineClient.Machines(clone.Namespace).UpdateStatus(ctx, clone, metav1.UpdateOptions{}); err != nil {
660662
return machineutils.ShortRetry, fmt.Errorf("failed to persist status addresses after initialization was successful: %w", err)

0 commit comments

Comments
 (0)