E2E: Fix log collection for machines

lentzi90 · lentzi90 · commit 6d3524b9c224 · 2025-10-24T08:39:06.000Z
I mistakenly removed the namespace when switching to CAPI v1beta2.
The infrastructureRef no longer specifies a namespace since it is implied to be
the same as the clusters namespace. However, we still need to set a namespace
for the NamespacedName...

Signed-off-by: Lennart Jern &lt;lennart.jern@est.tech&gt;
diff --git a/test/e2e/shared/common.go b/test/e2e/shared/common.go
@@ -169,7 +169,8 @@ func getOpenStackClusterFromMachine(ctx context.Context, client client.Client, m
 	}
 
 	key = types.NamespacedName{
-		Name: cluster.Spec.InfrastructureRef.Name,
+		Name:      cluster.Spec.InfrastructureRef.Name,
+		Namespace: cluster.Namespace,
 	}
 	openStackCluster := &infrav1.OpenStackCluster{}
 	err = client.Get(ctx, key, openStackCluster)

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,8 @@ func getOpenStackClusterFromMachine(ctx context.Context, client client.Client, m`
`169`	`169`	`}`
`170`	`170`
`171`	`171`	`key = types.NamespacedName{`
`172`		`- Name: cluster.Spec.InfrastructureRef.Name,`
	`172`	`+ Name: cluster.Spec.InfrastructureRef.Name,`
	`173`	`+ Namespace: cluster.Namespace,`
`173`	`174`	`}`
`174`	`175`	`openStackCluster := &infrav1.OpenStackCluster{}`
`175`	`176`	`err = client.Get(ctx, key, openStackCluster)`