Fix more lint warnings

Signed-off-by: Cosmin Cojocar <[email protected]>
kubernetes-sigs · Nov 16, 2023 · 465c3ba · 465c3ba
1 parent c38b1c0
commit 465c3ba
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/internal/pkg/webhooks/recording/recording.go b/internal/pkg/webhooks/recording/recording.go
@@ -335,10 +335,8 @@ func (p *podSeccompRecorder) setActiveWorkloads(
 	newActiveWorkloads := profileRecording.Status.ActiveWorkloads
 	if op == admissionv1.Delete {
 		newActiveWorkloads = utils.RemoveIfExists(newActiveWorkloads, podName)
-	} else {
-		if selector.Matches(podLabels) {
-			newActiveWorkloads = utils.AppendIfNotExists(newActiveWorkloads, podName)
-		}
+	} else if selector.Matches(podLabels) {
+		newActiveWorkloads = utils.AppendIfNotExists(newActiveWorkloads, podName)
 	}
 
 	profileRecording.Status.ActiveWorkloads = newActiveWorkloads
@@ -357,11 +355,9 @@ func (p *podSeccompRecorder) setFinalizers(
 		if controllerutil.ContainsFinalizer(profileRecording, finalizer) {
 			controllerutil.RemoveFinalizer(profileRecording, finalizer)
 		}
-	} else {
-		if selector.Matches(podLabels) {
-			if !controllerutil.ContainsFinalizer(profileRecording, finalizer) {
-				controllerutil.AddFinalizer(profileRecording, finalizer)
-			}
+	} else if selector.Matches(podLabels) {
+		if !controllerutil.ContainsFinalizer(profileRecording, finalizer) {
+			controllerutil.AddFinalizer(profileRecording, finalizer)
 		}
 	}