fix: remove logs of 'argv' and 'context.config' (#2378)

Signed-off-by: Zhan Milenkov <[email protected]>

Author: instamenta

src/commands/account.ts

@@ -277,8 +277,6 @@ export class AccountCommand extends BaseCommand {
             // set config in the context for later tasks to use
             context_.config = config;
 
-            self.logger.debug('Initialized config', {config});
-
             await self.accountManager.loadNodeClient(
               config.namespace,
               self.remoteConfig.getClusterRefs(),
@@ -521,8 +519,6 @@ export class AccountCommand extends BaseCommand {
             // set config in the context for later tasks to use
             context_.config = config;
 
-            self.logger.debug('Initialized config', {config});
-
             await self.accountManager.loadNodeClient(
               context_.config.namespace,
               self.remoteConfig.getClusterRefs(),
@@ -621,8 +617,6 @@ export class AccountCommand extends BaseCommand {
               config.deployment,
               self.configManager.getFlag<boolean>(flags.forcePortForward),
             );
-
-            self.logger.debug('Initialized config', {config});
           },
         },
         {
@@ -726,8 +720,6 @@ export class AccountCommand extends BaseCommand {
               config.deployment,
               self.configManager.getFlag<boolean>(flags.forcePortForward),
             );
-
-            self.logger.debug('Initialized config', {config});
           },
         },
         {

src/commands/block-node.ts

@@ -238,8 +238,6 @@ export class BlockNodeCommand extends BaseCommand {
 
             context_.config.context = this.remoteConfig.getClusterRefs()[context_.config.clusterRef];
 
-            this.logger.debug('Initialized config', {config: context_.config});
-
             return ListrLock.newAcquireLockTask(lease, task);
           },
         },
@@ -432,8 +430,6 @@ export class BlockNodeCommand extends BaseCommand {
               }
             }
 
-            this.logger.debug('Initialized config', {config: context_.config});
-
             return ListrLock.newAcquireLockTask(lease, task);
           },
         },
@@ -522,8 +518,6 @@ export class BlockNodeCommand extends BaseCommand {
 
             context_.config.context = this.remoteConfig.getClusterRefs()[context_.config.clusterRef];
 
-            this.logger.debug('Initialized config', {config: context_.config});
-
             return ListrLock.newAcquireLockTask(lease, task);
           },
         },

src/commands/cluster/configs.ts

@@ -110,8 +110,6 @@ export class ClusterCommandConfigs {
       clusterRef: configManager.getFlag<ClusterReferenceName>(flags.clusterRef),
     } as ClusterReferenceSetupConfigClass;
 
-    this.logger.debug('Prepare ctx.config', {config: context_.config, argv});
-
     context_.config.context =
       this.localConfig.configuration.clusterRefs.get(context_.config.clusterRef)?.toString() ??
       this.k8Factory.default().contexts().readCurrent();

src/commands/deployment.ts

@@ -144,8 +144,6 @@ export class DeploymentCommand extends BaseCommand {
             ) {
               throw new SoloError(ErrorMessages.DEPLOYMENT_NAME_ALREADY_EXISTS(context_.config.deployment));
             }
-
-            self.logger.debug('Prepared config', {config: context_.config, cachedConfig: self.configManager.config});
           },
         },
         {
@@ -228,8 +226,6 @@ export class DeploymentCommand extends BaseCommand {
             ) {
               throw new SoloError(ErrorMessages.DEPLOYMENT_NAME_ALREADY_EXISTS(context_.config.deployment));
             }
-
-            self.logger.debug('Prepared config', {config: context_.config, cachedConfig: self.configManager.config});
           },
         },
         {
@@ -335,13 +331,10 @@ export class DeploymentCommand extends BaseCommand {
             await self.localConfig.load();
 
             self.configManager.update(argv);
-            self.logger.debug('Updated config with argv', {config: self.configManager.config});
             await self.configManager.executePrompt(task, [flags.clusterRef]);
             context_.config = {
               clusterName: self.configManager.getFlag<ClusterReferenceName>(flags.clusterRef),
             } as Config;
-
-            self.logger.debug('Prepared config', {config: context_.config, cachedConfig: self.configManager.config});
           },
         },
         {
@@ -525,8 +518,6 @@ export class DeploymentCommand extends BaseCommand {
           nodeAliases: [] as NodeAliases,
           context: '',
         };
-
-        this.logger.debug('Prepared config', {config: context_.config, cachedConfig: this.configManager.config});
       },
     };
   }

src/commands/network.ts

@@ -652,7 +652,6 @@ export class NetworkCommand extends BaseCommand {
     argv: ArgvStruct,
   ): Promise<NetworkDeployConfigClass> {
     this.configManager.update(argv);
-    this.logger.debug('Updated config with argv', {config: this.configManager.config});
 
     const flagsWithDisabledPrompts: CommandFlag[] = [
       flags.apiPermissionProperties,
@@ -780,10 +779,6 @@ export class NetworkCommand extends BaseCommand {
     this.logger.debug('Preparing storage secrets');
     await this.prepareStorageSecrets(config);
 
-    this.logger.debug('Prepared config', {
-      config,
-      cachedConfig: this.configManager.config,
-    });
     return config;
   }
 

src/commands/node/tasks.ts

@@ -2773,8 +2773,6 @@ export class NodeCommandTasks {
           }
         }
 
-        this.logger.debug('Initialized config', {config});
-
         if (lease) {
           return ListrLock.newAcquireLockTask(lease, task);
         }

src/commands/relay.ts

@@ -321,8 +321,6 @@ export class RelayCommand extends BaseCommand {
               }
             }
 
-            self.logger.debug('Initialized config', {config: context_.config});
-
             return ListrLock.newAcquireLockTask(lease, task);
           },
         },
@@ -531,8 +529,6 @@ export class RelayCommand extends BaseCommand {
               context_.config.context,
             );
 
-            self.logger.debug('Initialized config', {config: context_.config});
-
             return ListrLock.newAcquireLockTask(lease, task);
           },
         },
@@ -584,7 +580,7 @@ export class RelayCommand extends BaseCommand {
               flags.setOptionalCommandFlags(y, ...RelayCommand.DEPLOY_FLAGS_LIST.optional);
             },
             handler: async (argv: ArgvStruct) => {
-              self.logger.info("==== Running 'relay deploy' ===", {argv});
+              self.logger.info("==== Running 'relay deploy' ===");
 
               await self.deploy(argv).then(r => {
                 self.logger.info('==== Finished running `relay deploy`====');
@@ -602,8 +598,7 @@ export class RelayCommand extends BaseCommand {
               flags.setOptionalCommandFlags(y, ...RelayCommand.DESTROY_FLAGS_LIST.optional);
             },
             handler: async (argv: ArgvStruct) => {
-              self.logger.info("==== Running 'relay destroy' ===", {argv});
-              self.logger.debug(argv);
+              self.logger.info("==== Running 'relay destroy' ===");
 
               await self.destroy(argv).then(r => {
                 self.logger.info('==== Finished running `relay destroy`====');