Merge pull request #12 from projectsyn/feat/better-log-messages

Clean up log messages related to Lieutenant API
projectsyn · Jan 14, 2022 · ef529ac · ef529ac
2 parents 44c5313 + 3a1a592
commit ef529ac
Showing 1 changed file with 23 additions and 4 deletions.
diff --git a/src/commodore/index.ts b/src/commodore/index.ts
@@ -129,20 +129,39 @@ export async function extractPackageFile(
     if (config.lieutenantURL && config.lieutenantURL != '') {
       if (config.lieutenantToken == '') {
         logger.warn(
-          `Lieutenant token is empty. Renovate won't try to query the Lieutenant API at ${config.lieutenantURL}`
+          {
+            cluster: cluster.name,
+            lieutenantURL: config.lieutenantURL,
+          },
+          "Lieutenant token is empty. Renovate won't try to query the Lieutenant API"
         );
       } else {
-        logger.info(`Querying Lieutenant at ${config.lieutenantURL}`);
+        logger.debug(
+          {
+            lieutenantURL: config.lieutenantURL,
+          },
+          'Querying Lieutenant'
+        );
         try {
           clusterInfo = await fetchClusterInfo(config, cluster.name);
         } catch (error: any) {
           if (error instanceof LieutenantError) {
             const err = error as LieutenantError;
             if (err.statusCode == 404) {
-              logger.debug(`Lieutenant query returned 404 for ${cluster.name}`);
+              logger.debug(
+                {
+                  cluster: cluster.name,
+                },
+                'Lieutenant query returned 404'
+              );
             } else {
               logger.info(
-                `Error querying Lieutenant for ${cluster.name}: statusCode=${err.statusCode}, reason=${err.message}`
+                {
+                  cluster: cluster.name,
+                  statusCode: err.statusCode,
+                  reason: err.message,
+                },
+                'Error querying Lieutenant'
               );
             }
           } else {