Skip to content

Commit 7e1f658

Browse files
authored
Merge pull request #9191 from chaunceyjiang/topology-owned
✨ Add topology-owned label to MachineHealthChecks.
2 parents c27d755 + 5ca00ab commit 7e1f658

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

internal/controllers/topology/cluster/desired_state.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1001,6 +1001,9 @@ func computeMachineHealthCheck(healthCheckTarget client.Object, selector *metav1
10011001
ObjectMeta: metav1.ObjectMeta{
10021002
Name: healthCheckTarget.GetName(),
10031003
Namespace: healthCheckTarget.GetNamespace(),
1004+
Labels: map[string]string{
1005+
clusterv1.ClusterTopologyOwnedLabel: "",
1006+
},
10041007
},
10051008
Spec: clusterv1.MachineHealthCheckSpec{
10061009
ClusterName: clusterName,

internal/controllers/topology/cluster/desired_state_test.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2246,7 +2246,10 @@ func Test_computeMachineHealthCheck(t *testing.T) {
22462246
Name: "md1",
22472247
Namespace: "ns1",
22482248
// Label is added by defaulting values using MachineHealthCheck.Default()
2249-
Labels: map[string]string{"cluster.x-k8s.io/cluster-name": "cluster1"},
2249+
Labels: map[string]string{
2250+
"cluster.x-k8s.io/cluster-name": "cluster1",
2251+
clusterv1.ClusterTopologyOwnedLabel: "",
2252+
},
22502253
},
22512254
Spec: clusterv1.MachineHealthCheckSpec{
22522255
ClusterName: "cluster1",

0 commit comments

Comments
 (0)