From 0ea236b8b0737c887fe36378ecf1793bba8e9bc7 Mon Sep 17 00:00:00 2001
From: Ken Finnigan <ken@kenfinnigan.me>
Date: Wed, 6 Oct 2021 13:55:49 -0400
Subject: [PATCH] Remove debugging statements

---
 src/main/java/com/redhat/MeterController.java | 22 +++----------------
 1 file changed, 3 insertions(+), 19 deletions(-)

diff --git a/src/main/java/com/redhat/MeterController.java b/src/main/java/com/redhat/MeterController.java
index 86d39a1..ef315a5 100644
--- a/src/main/java/com/redhat/MeterController.java
+++ b/src/main/java/com/redhat/MeterController.java
@@ -1,7 +1,6 @@
 package com.redhat;
 
 import java.util.Map;
-import java.util.concurrent.atomic.AtomicBoolean;
 
 import org.eclipse.microprofile.config.inject.ConfigProperty;
 import org.jboss.logging.Logger;
@@ -110,7 +109,6 @@ public UpdateControl<Meter> createOrUpdateResource(Meter resource, Context<Meter
 
     void createServiceMonitor() {
         if (serviceMonitor().get() != null) {
-            System.out.println("SKIPPED INSTALL");
             // Don't repeat the install
             return;
         }
@@ -129,18 +127,13 @@ void createServiceMonitor() {
                     .build();
             role = client.rbac().roles().createOrReplace(role);
 
-            // TODO remove
-            System.out.println("ROLE RESOURCE VERSION: " + role.getMetadata().getResourceVersion());
-            System.out.println("ROLE NAMESPACE: " + role.getMetadata().getNamespace());
-            System.out.println("ROLE NAME: " + role.getMetadata().getName());
             if (role.getMetadata().getResourceVersion() == null) {
                 // No exception, but Role not created properly
                 return;
             }
         } catch (Exception e) {
             LOG.error("Failed to create Role", e);
-            //TODO Uncomment when permissions are fixed
-//            return;
+            return;
         }
 
         try {
@@ -161,17 +154,13 @@ void createServiceMonitor() {
                     .build();
             roleBinding = client.rbac().roleBindings().createOrReplace(roleBinding);
 
-            // TODO remove
-            System.out.println("ROLEBINDING RESOURCE VERSION: " + roleBinding.getMetadata().getResourceVersion());
-
             if (roleBinding.getMetadata().getResourceVersion() == null) {
                 // No exception, but RoleBinding not created properly
                 return;
             }
         } catch (Exception e) {
             LOG.error("Failed to create RoleBinding", e);
-            //TODO Uncomment when permissions are fixed
-//            return;
+            return;
         }
 
         try {
@@ -193,16 +182,13 @@ void createServiceMonitor() {
                     .build();
             promRule = client.monitoring().prometheusRules().inNamespace(OPENSHIFT_MONITORING_NAMESPACE).createOrReplace(promRule);
 
-            // TODO remove
-            System.out.println("PROMETHEUSRULE RESOURCE VERSION: " + promRule.getMetadata().getResourceVersion());
             if (promRule.getMetadata().getResourceVersion() == null) {
                 // No exception, but PrometheusRule not created properly
                 return;
             }
         } catch (Exception e) {
             LOG.error("Failed to create PrometheusRule", e);
-            //TODO Uncomment when permissions are fixed
-            // return;
+            return;
         }
 
         try {
@@ -228,8 +214,6 @@ void createServiceMonitor() {
                     .endSpec()
                     .build();
             monitor = client.monitoring().serviceMonitors().inNamespace(OPENSHIFT_MONITORING_NAMESPACE).createOrReplace(monitor);
-            // TODO remove
-            System.out.println("SERVICEMONITOR RESOURCE VERSION: " + monitor.getMetadata().getResourceVersion());
 
             if (!monitor.getMetadata().getResourceVersion().isEmpty()) {
                 LOG.info("ServiceMonitor " + SERVICE_MONITOR_NAME + " installed.");