Do not delete cluster that is older than 7 days (#88)

* do-not-delete-clusters-that-are-older-than-7-days * typos * Update controllers/cluster_controller.go Co-authored-by: Andreas Sommer <[email protected]> * Update CHANGELOG.md Co-authored-by: Andreas Sommer <[email protected]> --------- Co-authored-by: Andreas Sommer <[email protected]>
giantswarm · Apr 30, 2024 · 357c976 · 357c976
1 parent c6154db
commit 357c976
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- Ignore cluster from being deleted, if it is older than 7 days and does not have the `keep-until` label. This is to prevent the deletion of customer clusters in case this app is accidentally deployed to a production MC.
+
 ## [0.9.0] - 2024-02-15
 
 ### Changed

diff --git a/controllers/cluster_controller.go b/controllers/cluster_controller.go
@@ -102,6 +102,14 @@ func (r *ClusterReconciler) reconcile(ctx context.Context, cluster *capi.Cluster
 			log.Info(fmt.Sprintf("Found label %s. Cluster will be ignored for deletion", keepUntil))
 			return ctrl.Result{RequeueAfter: 24 * time.Hour}, nil
 		}
+	} else {
+		// ignore cluster from being deleted if it is older than 7 days and do NOT have keep-until label
+		// this is to prevent deletion in a case of accidental deployment of the app to production MCs
+		if time.Since(cluster.CreationTimestamp.Time).Hours() > 24*7 {
+			log.Info(fmt.Sprintf("Cluster is older than 7 days and does not have label %s. Cluster will be ignored for deletion", keepUntil))
+			return ctrl.Result{}, nil
+		}
+
 	}
 
 	// immediately delete the cluster if defaultTTL has passed