OKAPI-1172: Reduce excessive logging on deploy

When Okapi deploys a module it logs the complete module descriptor. This is too verbose. It is not needed because the information is available with `docker inspect`.
folio-org · Sep 13, 2023 · 76dd5e8 · 76dd5e8
1 parent 7d26d90
commit 76dd5e8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/okapi-core/src/main/java/org/folio/okapi/managers/DeploymentManager.java b/okapi-core/src/main/java/org/folio/okapi/managers/DeploymentManager.java
@@ -166,7 +166,7 @@ private Future<DeploymentDescriptor> deploy2(int usePort, DeploymentDescriptor m
         }
         descriptor.setEnv(nenv);
       }
-      logger.info("Deploy {} {}", md.getSrvcId(), Json.encodePrettily(descriptor));
+      logger.info("Deploy {} {}", md.getSrvcId(), md.getDescriptor().getDockerImage());
       String moduleUrl = "http://" + host + ":" + usePort;
       String moduleHost = host;
       if (descriptor.getDockerImage() != null) {

diff --git a/okapi-core/src/main/java/org/folio/okapi/managers/DiscoveryManager.java b/okapi-core/src/main/java/org/folio/okapi/managers/DiscoveryManager.java
@@ -153,9 +153,9 @@ Future<DeploymentDescriptor> addAndDeployIgnoreError(DeploymentDescriptor dd) {
    * </ol>
    */
   private Future<DeploymentDescriptor> addAndDeploy0(DeploymentDescriptor dd) {
-    String tmp = Json.encodePrettily(dd);
-    logger.info("addAndDeploy: {}", tmp);
     final String id = dd.getSrvcId();
+    logger.info("addAndDeploy: {} {} {}", dd.getNodeId(), id,
+        dd.getDescriptor() == null ? null : dd.getDescriptor().getDockerImage());
     if (id == null) {
       return Future.failedFuture(new OkapiError(ErrorType.USER, messages.getMessage("10800")));
     }