Merge pull request #1806 from k8s-infra-cherrypick-robot/cherry-pick-1803-to-release-0.9

k8s-ci-robot · web-flow · commit 95a4004861db · 2024-01-02T17:48:58.000+01:00
[release-0.9] 🐛 Fix potential panic during instance create
diff --git a/pkg/cloud/services/compute/instance.go b/pkg/cloud/services/compute/instance.go
@@ -348,7 +348,7 @@ func (s *Service) createInstanceImpl(eventObject runtime.Object, openStackCluste
 		return createdInstance.State() == infrav1.InstanceStateActive, nil
 	})
 	if err != nil {
-		record.Warnf(eventObject, "FailedCreateServer", "Failed to create server %s: %v", createdInstance.Name(), err)
+		record.Warnf(eventObject, "FailedCreateServer", "Failed to create server %s: %v", instanceSpec.Name, err)
 		return nil, err
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,7 @@ func (s *Service) createInstanceImpl(eventObject runtime.Object, openStackCluste`
`348`	`348`	`return createdInstance.State() == infrav1.InstanceStateActive, nil`
`349`	`349`	`})`
`350`	`350`	`if err != nil {`
`351`		`- record.Warnf(eventObject, "FailedCreateServer", "Failed to create server %s: %v", createdInstance.Name(), err)`
	`351`	`+ record.Warnf(eventObject, "FailedCreateServer", "Failed to create server %s: %v", instanceSpec.Name, err)`
`352`	`352`	`return nil, err`
`353`	`353`	`}`
`354`	`354`