fixup! feat: add DevWorkspace pruner

Signed-off-by: Oleksii Kurinnyi <[email protected]>

Author: akurinnoy

controllers/workspace/pruner_controller.go

@@ -1,10 +1,22 @@
+// Copyright (c) 2019-2024 Red Hat, Inc.
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 package controllers
 
 import (
 	"context"
 	"fmt"
 	"strconv"
-	"time"
 
 	controllerv1alpha1 "github.com/devfile/devworkspace-operator/apis/controller/v1alpha1"
 	"github.com/devfile/devworkspace-operator/pkg/constants"
@@ -93,14 +105,12 @@ func (r *PrunerReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctr
 		return ctrl.Result{}, err
 	}
 
-	return ctrl.Result{RequeueAfter: 24 * time.Hour}, nil
+	return ctrl.Result{}, nil
 }
 
 func (r *PrunerReconciler) SetupWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewControllerManagedBy(mgr).
 		For(&controllerv1alpha1.DevWorkspaceOperatorConfig{}).
-		// Owns(&batchv1.CronJob{}).
-		// Owns(&corev1.ConfigMap{}).
 		Complete(r)
 }
 
@@ -319,13 +329,14 @@ func prunerDefaultConfigMap(namespace string) *corev1.ConfigMap {
 current_time=$(date +%s)
 echo "Current time: ${current_time}"
 echo "RETAIN_TIME: ${RETAIN_TIME}"
-for namespace in $(oc get namespaces -o go-template='{{range .items}}{{.metadata.name}}{{"\n"}}{{end}}' | grep ".-devspaces")
+for namespace in $(oc get namespaces -l app.kubernetes.io/component=workspaces-namespace -o go-template='{{range .items}}{{.metadata.name}}{{"\n"}}{{end}}')
 do
   for workspace in $(oc get devworkspaces -n ${namespace} -o go-template='{{range .items}}{{.metadata.name}}{{"\n"}}{{end}}')
   do
     last_start=$(date -d$(oc get devworkspace ${workspace} -n ${namespace} -o go-template='{{range .status.conditions}}{{if eq .type "Started"}}{{.lastTransitionTime}}{{end}}{{end}}') +%s)
     workspace_age=$(( ${current_time} - ${last_start} ))
-    if [[ ${workspace_age} -gt ${RETAIN_TIME} ]]
+    started=$(oc get devworkspace ${workspace} -n ${namespace} -o go-template='{{.spec.started}}')
+    if [[ "$started" != "true" ]] && [[ ${workspace_age} -gt ${RETAIN_TIME} ]]
     then
       echo "Removing workspace: ${workspace} in ${namespace}"
       oc delete devworkspace ${workspace} -n ${namespace}