Reinstate cleanup, remove debug printlns.

k8ssandra · Oct 19, 2023 · 785fc1b · 785fc1b
1 parent a05fed1
commit 785fc1b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/controllers/reaper/reaper_controller.go b/controllers/reaper/reaper_controller.go
@@ -159,7 +159,6 @@ func (r *ReaperReconciler) reconcileDeployment(
 	deploymentKey := types.NamespacedName{Namespace: actualReaper.Namespace, Name: actualReaper.Name}
 	logger = logger.WithValues("Deployment", deploymentKey)
 	logger.Info(fmt.Sprintf("Reconciling reaper deployment, req was %#v", actualReaper))
-	println("reaper http management proxy was: ", actualReaper.Spec.HttpManagement.Enabled)
 
 	authVars, err := r.collectAuthVars(ctx, actualReaper, logger)
 	if err != nil {

diff --git a/pkg/reaper/deployment.go b/pkg/reaper/deployment.go
@@ -152,9 +152,7 @@ func NewDeployment(reaper *api.Reaper, dc *cassdcapi.CassandraDatacenter, keysto
 			Value: fmt.Sprintf("%d", reaper.Spec.HeapSize.Value()),
 		})
 	}
-	println("Reconciling new Reaper deployment: ", reaper)
 	if reaper.Spec.HttpManagement.Enabled {
-		println("Reconciling new Reaper deployment,HttpManagement.Enabled ")
 		envVars = append(envVars, corev1.EnvVar{
 			Name:  "REAPER_HTTP_MANAGEMENT_ENABLE",
 			Value: "true",

diff --git a/test/e2e/suite_test.go b/test/e2e/suite_test.go
@@ -249,8 +249,8 @@ func TestOperator(t *testing.T) {
 	t.Run("CreateSingleReaper", e2eTest(ctx, &e2eTestOpts{
 		testFunc:                     createSingleReaper,
 		fixture:                      framework.NewTestFixture("single-dc-reaper", controlPlane),
-		skipK8ssandraClusterCleanup:  true,  // TODO: reinstate cleanup
-		doCassandraDatacenterCleanup: false, // TODO: reinstate cleanup
+		skipK8ssandraClusterCleanup:  false,
+		doCassandraDatacenterCleanup: true,
 	}))
 	t.Run("CreateMultiReaper", e2eTest(ctx, &e2eTestOpts{
 		testFunc: createMultiReaper,