From 896ee8d1832233c745993ee9a109c2fb934b7103 Mon Sep 17 00:00:00 2001 From: Jean-Philippe Evrard Date: Mon, 8 Jun 2026 15:15:55 +0200 Subject: [PATCH] chore: Quick fix to pass linters This whole block should be refactored, but this will be done inside v2. At the same time, it should not block the golangci-lint, so I did exactly what the golangci lint wanted to have. Signed-off-by: Jean-Philippe Evrard --- pkg/taints/taints.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/pkg/taints/taints.go b/pkg/taints/taints.go index 54a0441..8ae039c 100644 --- a/pkg/taints/taints.go +++ b/pkg/taints/taints.go @@ -15,6 +15,8 @@ import ( "k8s.io/client-go/kubernetes" ) +const addOrRemoveJSONOp = "add" + // Taint allows to set soft and hard limitations for scheduling and executing pods on nodes. type Taint struct { client *kubernetes.Clientset @@ -107,7 +109,6 @@ func preferNoSchedule(client *kubernetes.Clientset, nodeID, taintName string, ef } var patches []patchTaints - if len(updatedNode.Spec.Taints) == 0 { // add first taint and ensure to keep current taints patches = []patchTaints{ @@ -117,12 +118,12 @@ func preferNoSchedule(client *kubernetes.Clientset, nodeID, taintName string, ef Value: updatedNode.Spec, }, { - Op: "add", + Op: addOrRemoveJSONOp, Path: "/spec/taints", Value: []v1.Taint{}, }, { - Op: "add", + Op: addOrRemoveJSONOp, Path: "/spec/taints/-", Value: taint, }, @@ -144,7 +145,7 @@ func preferNoSchedule(client *kubernetes.Clientset, nodeID, taintName string, ef // add missing taint to exsting list patches = []patchTaints{ { - Op: "add", + Op: addOrRemoveJSONOp, Path: "/spec/taints/-", Value: taint, },