This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new ddf53ca136 HDDS-10271. Reduce verbosity of some SCM start-up logs
(#6146)
ddf53ca136 is described below
commit ddf53ca136825af77ab54baaf75c06c1b74218f7
Author: Nandakumar Vadivelu <[email protected]>
AuthorDate: Fri Feb 2 23:07:52 2024 +0530
HDDS-10271. Reduce verbosity of some SCM start-up logs (#6146)
---
.../security/x509/certificate/client/DefaultCertificateClient.java | 5 ++++-
.../apache/hadoop/hdds/scm/container/ContainerStateManagerImpl.java | 4 ++--
.../org/apache/hadoop/hdds/scm/server/StorageContainerManager.java | 1 -
3 files changed, 6 insertions(+), 4 deletions(-)
diff --git
a/hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/security/x509/certificate/client/DefaultCertificateClient.java
b/hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/security/x509/certificate/client/DefaultCertificateClient.java
index 09f27e46db..d3db81c71b 100644
---
a/hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/security/x509/certificate/client/DefaultCertificateClient.java
+++
b/hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/security/x509/certificate/client/DefaultCertificateClient.java
@@ -245,8 +245,11 @@ public abstract class DefaultCertificateClient implements
CertificateClient {
updateCachedData(fileName, CAType.SUBORDINATE,
this::updateCachedSubCAId);
updateCachedData(fileName, CAType.ROOT, this::updateCachedRootCAId);
- getLogger().info("Added certificate {} from file: {}.", cert,
+ getLogger().info("Added certificate {} from file: {}.", readCertSerialId,
filePath.toAbsolutePath());
+ if (getLogger().isDebugEnabled()) {
+ getLogger().debug("Certificate: {}", cert);
+ }
} catch (java.security.cert.CertificateException
| IOException | IndexOutOfBoundsException e) {
getLogger().error("Error reading certificate from file: {}.",
diff --git
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerStateManagerImpl.java
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerStateManagerImpl.java
index 62e1f01935..cf5975d05e 100644
---
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerStateManagerImpl.java
+++
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerStateManagerImpl.java
@@ -255,8 +255,8 @@ public final class ContainerStateManagerImpl
// CLOSING state by ReplicationManager's OpenContainerHandler
// For more info: HDDS-10231
LOG.warn("Found container {} which is in OPEN state with " +
- "pipeline {} that does not exist. Marking container for " +
- "closing.", container, container.getPipelineID());
+ "pipeline {} that does not exist.",
+ container, container.getPipelineID());
}
}
}
diff --git
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/StorageContainerManager.java
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/StorageContainerManager.java
index 046be68760..11fdc0d16d 100644
---
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/StorageContainerManager.java
+++
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/StorageContainerManager.java
@@ -450,7 +450,6 @@ public final class StorageContainerManager extends
ServiceRuntimeInfoImpl
moveManager = new MoveManager(replicationManager, containerManager);
containerReplicaPendingOps.registerSubscriber(moveManager);
containerBalancer = new ContainerBalancer(this);
- LOG.info(containerBalancer.toString());
// Emit initial safe mode status, as now handlers are registered.
scmSafeModeManager.emitSafeModeStatus();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
