Skip to content

Commit db8a896

Browse files
committed
remove enabling instance-ha on reconcile eviction
1 parent 21fc4b1 commit db8a896

2 files changed

Lines changed: 0 additions & 8 deletions

File tree

internal/controller/node_eviction_label_controller.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -140,10 +140,6 @@ func (r *NodeEvictionLabelReconciler) reconcileEviction(ctx context.Context, evi
140140
Reason: fmt.Sprintf("openstack-hypervisor-operator: label %v=%v", labelEvictionRequired, maintenanceValue),
141141
}
142142

143-
if err = enableInstanceHAMissingOkay(node); err != nil {
144-
return "", fmt.Errorf("failed to enable instance ha before eviction due to %w", err)
145-
}
146-
147143
if err = r.Create(ctx, eviction); err != nil {
148144
return "", fmt.Errorf("failed to create eviction due to %w", err)
149145
}

internal/controller/utils.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,6 @@ func enableInstanceHA(node *corev1.Node) error {
7979
return updateInstanceHA(node, `{"enabled": true}`, []int{http.StatusOK})
8080
}
8181

82-
func enableInstanceHAMissingOkay(node *corev1.Node) error {
83-
return updateInstanceHA(node, `{"enabled": true}`, []int{http.StatusOK, http.StatusNotFound})
84-
}
85-
8682
func disableInstanceHA(node *corev1.Node) error {
8783
return updateInstanceHA(node, `{"enabled": false}`, []int{http.StatusOK, http.StatusNotFound})
8884
}

0 commit comments

Comments
 (0)