Skip to content

Commit 52f840e

Browse files
authored
Fix panic if provider does not support Driver.InitializeMachine (#1032)
1 parent 32bd76c commit 52f840e

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
@@ -649,7 +649,9 @@ func (c *controller) triggerCreationFlow(ctx context.Context, createMachineReque
649649
if c.targetCoreClient == nil {
650650
// persist addresses from the InitializeMachine and CreateMachine responses
651651
clone := clone.DeepCopy()
652-
addresses.Insert(initResponse.Addresses...)
652+
if initResponse != nil {
653+
addresses.Insert(initResponse.Addresses...)
654+
}
653655
clone.Status.Addresses = buildAddressStatus(addresses, nodeName)
654656
if _, err := c.controlMachineClient.Machines(clone.Namespace).UpdateStatus(ctx, clone, metav1.UpdateOptions{}); err != nil {
655657
return machineutils.ShortRetry, fmt.Errorf("failed to persist status addresses after initialization was successful: %w", err)

0 commit comments

Comments
 (0)