Skip to content

Commit ec83410

Browse files
committed
fix panic dereferencing
Signed-off-by: MatthieuFin <[email protected]>
1 parent 3a3a53b commit ec83410

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

controllers/openstackmachine_controller.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -422,7 +422,11 @@ func (r *OpenStackMachineReconciler) reconcileMachineState(scope *scope.WithLogg
422422
conditions.MarkTrue(openStackMachine, infrav1.InstanceReadyCondition)
423423

424424
// Set properties required by CAPI machine controller
425-
openStackMachine.Spec.ProviderID = ptr.To(fmt.Sprintf("openstack://%s/%s", openStackMachine.Spec.IdentityRef.Region, *openStackServer.Status.InstanceID))
425+
var region string
426+
if openStackMachine.Spec.IdentityRef != nil {
427+
region = openStackMachine.Spec.IdentityRef.Region
428+
}
429+
openStackMachine.Spec.ProviderID = ptr.To(fmt.Sprintf("openstack://%s/%s", region, *openStackServer.Status.InstanceID))
426430
openStackMachine.Status.InstanceID = openStackServer.Status.InstanceID
427431
openStackMachine.Status.Ready = true
428432
case infrav1.InstanceStateError:

0 commit comments

Comments
 (0)