Remove redundant helper methods

Signed-off-by: Kate Goldenring <[email protected]>

Author: kate-goldenring

internal/containerd/install_dbus.go

@@ -3,20 +3,8 @@ package containerd
 import (
 	"fmt"
 	"log/slog"
-	"os"
-	"os/exec"
 )
 
-// UsesSystemd checks if the system is using systemd.
-func UsesSystemd() bool {
-	cmd := nsenterCmd("systemctl", "list-units", "|", "grep", "-q", "containerd.service")
-	if err := cmd.Run(); err != nil {
-		slog.Info("Error with systemctl: %w\n", "error", err)
-		return false
-	}
-	return true
-}
-
 // InstallDbus checks if D-Bus service is installed and active. If not, installs D-Bus
 // and starts the service.
 // NOTE: this limits support to systems using systemctl to manage systemd.
@@ -63,8 +51,3 @@ func InstallDbus() error {
 
 	return nil
 }
-
-func nsenterCmd(cmd ...string) *exec.Cmd {
-	return exec.Command("nsenter",
-		append([]string{fmt.Sprintf("-m/%s/proc/1/ns/mnt", os.Getenv("HOST_ROOT")), "--"}, cmd...)...) // #nosec G204
-}

internal/controller/shim_controller.go

@@ -380,6 +380,8 @@ func (sr *ShimReconciler) setOperationConfiguration(shim *rcmv1.Shim, opConfig *
 }
 
 // createJobManifest creates a Job manifest for a Shim.
+//
+//nolint:funlen // function is longer due to scaffolding an entire K8s Job manifest
 func (sr *ShimReconciler) createJobManifest(shim *rcmv1.Shim, node *corev1.Node, operation string) (*batchv1.Job, error) {
 	opConfig := opConfig{
 		operation:  operation,