Merge pull request #10971 from vincepri/bug-fix-cluster-proxy

k8s-ci-robot · web-flow · commit ab7bac8bc0b2 · 2024-07-31T10:56:29.000-07:00
🐛 test/framework isDockerCluster should check that infra ref is present
diff --git a/test/framework/cluster_proxy.go b/test/framework/cluster_proxy.go
@@ -478,7 +478,7 @@ func (p *clusterProxy) isDockerCluster(ctx context.Context, namespace string, na
 		return cl.Get(ctx, key, cluster)
 	}, retryableOperationTimeout, retryableOperationInterval).Should(Succeed(), "Failed to get %s", key)
 
-	return cluster.Spec.InfrastructureRef.Kind == "DockerCluster"
+	return cluster.Spec.InfrastructureRef != nil && cluster.Spec.InfrastructureRef.Kind == "DockerCluster"
 }
 
 func (p *clusterProxy) fixConfig(ctx context.Context, name string, config *api.Config) {

Original file line number	Diff line number	Diff line change
`@@ -478,7 +478,7 @@ func (p *clusterProxy) isDockerCluster(ctx context.Context, namespace string, na`
`478`	`478`	`return cl.Get(ctx, key, cluster)`
`479`	`479`	`}, retryableOperationTimeout, retryableOperationInterval).Should(Succeed(), "Failed to get %s", key)`
`480`	`480`
`481`		`- return cluster.Spec.InfrastructureRef.Kind == "DockerCluster"`
	`481`	`+ return cluster.Spec.InfrastructureRef != nil && cluster.Spec.InfrastructureRef.Kind == "DockerCluster"`
`482`	`482`	`}`
`483`	`483`
`484`	`484`	`func (p clusterProxy) fixConfig(ctx context.Context, name string, config api.Config) {`