Move some messages to CONFIG level to make log less chatty (#2387)

Author: rjeberhard

operator/src/main/java/oracle/kubernetes/operator/TuningParametersImpl.java

@@ -95,9 +95,9 @@ private void update() {
       if (!main.equals(this.main)
           || !callBuilder.equals(this.callBuilder)
           || !watch.equals(this.watch)
-          || !watch.equals(this.pod)
-          || !pod.equals(this.featureGates)) {
-        LOGGER.info(MessageKeys.TUNING_PARAMETERS);
+          || !pod.equals(this.pod)
+          || !featureGates.equals(this.featureGates)) {
+        LOGGER.config(MessageKeys.TUNING_PARAMETERS);
       }
       this.main = main;
       this.callBuilder = callBuilder;

operator/src/main/java/oracle/kubernetes/operator/utils/Certificates.java

@@ -51,7 +51,7 @@ private static String getCertificate(String path, String failureMessage) {
     try {
       return new String(Files.readAllBytes(GET_PATH.apply(path)));
     } catch (IOException e) {
-      LOGGER.warning(failureMessage, path);
+      LOGGER.config(failureMessage, path);
       return null;
     }
   }

operator/src/test/java/oracle/kubernetes/operator/utils/CertificatesTest.java

@@ -17,6 +17,7 @@
 
 import static oracle.kubernetes.operator.logging.MessageKeys.NO_EXTERNAL_CERTIFICATE;
 import static oracle.kubernetes.operator.logging.MessageKeys.NO_INTERNAL_CERTIFICATE;
+import static oracle.kubernetes.utils.LogMatcher.containsConfig;
 import static oracle.kubernetes.utils.LogMatcher.containsWarning;
 import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.notNullValue;
@@ -76,10 +77,10 @@ public void whenNoExternalCertificateFile_returnNull() {
   }
 
   @Test
-  public void whenNoExternalCertificateFile_logWarningMessage() {
+  public void whenNoExternalCertificateFile_logConfigMessage() {
     assertThat(Certificates.getOperatorExternalCertificateData(), nullValue());
 
-    assertThat(logRecords, containsWarning(NO_EXTERNAL_CERTIFICATE));
+    assertThat(logRecords, containsConfig(NO_EXTERNAL_CERTIFICATE));
   }
 
   @Test
@@ -97,10 +98,10 @@ public void whenNoInternalCertificateFile_returnNull() {
   }
 
   @Test
-  public void whenNoInternalCertificateFile_logWarningMessage() {
+  public void whenNoInternalCertificateFile_logConfigMessage() {
     Certificates.getOperatorInternalCertificateData();
 
-    assertThat(logRecords, containsWarning(NO_INTERNAL_CERTIFICATE));
+    assertThat(logRecords, containsConfig(NO_INTERNAL_CERTIFICATE));
   }
 
   @Test

operator/src/test/java/oracle/kubernetes/utils/LogMatcher.java

@@ -52,6 +52,10 @@ public static LogMatcher containsSevere(String expectedMessage) {
     return new LogMatcher(Level.SEVERE, expectedMessage);
   }
 
+  public static LogMatcher containsConfig(String expectedMessage) {
+    return new LogMatcher(Level.CONFIG, expectedMessage);
+  }
+
   public static LogMatcher containsFine(String expectedMessage) {
     return new LogMatcher(Level.FINE, expectedMessage);
   }