Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ledger Api Server logging: Lower selected logger.info to logger.debug #2641

Closed
oliverse-da opened this issue Aug 23, 2019 · 2 comments
Closed
Assignees
Labels
component/ledger Sandbox and Ledger API multi-domain Pertaining to multi-domain ledger integration

Comments

@oliverse-da
Copy link
Contributor

oliverse-da commented Aug 23, 2019

StandaloneIndexerServer:

-                  () => logger.info("successfully finished processing state updates")))(DEC)
+                  () => logger.debug("successfully finished processing state updates")))(DEC)
..
-    logger.info("Started Indexer Server")
+    logger.debug("Started Indexer Server")

StandaloneIndexServer:

-    logger.info("Initialized index server version {} with ledger-id = {}, port = {}, dar file = {}",
+    logger.debug("Initialized index server version {} with ledger-id = {}, port = {}, dar file = {}",
...
-    logger.info("Started Index Server")
+    logger.debug("Started Index Server")

JdbcIndexer:

-          logger.info(s"Initializing ledger with id: ${ledgerId.unwrap}")
+          logger.debug(s"Initializing ledger with id: ${ledgerId.unwrap}")

ReadOnlySqlLedger:

-    logger.info(s"Found existing ledger with id: ${foundLedgerId.unwrap}")
+    logger.debug(s"Found existing ledger with id: ${foundLedgerId.unwrap}")

FlywayMigrations:

-      logger.info(s"running Flyway migration..")
+      logger.debug(s"running Flyway migration..")
...
-      logger.info(s"Flyway schema migration finished successfully applying ${stepsTaken} steps.")
+      logger.debug(s"Flyway schema migration finished successfully applying ${stepsTaken} steps.")

LedgerApiServer:

        logger.info("Starting plainText server")
...
        logger.info("Starting TLS server")
...
      logger.info(s"listening on ${address.getOrElse("localhost")}:${grpcServer.getPort}")
@oliverse-da oliverse-da added component/ledger Sandbox and Ledger API multi-domain Pertaining to multi-domain ledger integration labels Aug 23, 2019
@oliverse-da oliverse-da added this to the Backlog milestone Aug 23, 2019
@oliverse-da oliverse-da self-assigned this Aug 23, 2019
@oliverse-da
Copy link
Contributor Author

logback setting workaround

@oliverse-da
Copy link
Contributor Author

Put in place canton work-around - closing as we discussed having logging useful for production rather than testing is the way to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ledger Sandbox and Ledger API multi-domain Pertaining to multi-domain ledger integration
Projects
None yet
Development

No branches or pull requests

1 participant