add check for loki config value in module to avoid loki initialization failure when config is not provided

Author: sanketpathak

workspaces/orchestrator/app-config.yaml

@@ -161,10 +161,9 @@ orchestrator:
   # Uncomment to set the content length limit for the requests. Defaults to 102400 bytes (100kb)
   # contentLengthLimit: 10mb
   # Uncomment and configure to use the log viewer
-  workflowLogProvider:
-    loki:
-      baseUrl: http://localhost:3100
-      token: secret-token
+  # workflowLogProvider:
+  #   loki:
+  #     baseUrl: http://localhost:3100
   # Uncomment if using a custom log stream selector
   # logStreamSelectors:
   #   - label: 'selector'

workspaces/orchestrator/plugins/orchestrator-backend-module-loki/src/module.ts

@@ -36,6 +36,12 @@ export const orchestratorModuleLoki = createBackendModule({
         workflowLogs: workflowLogsExtensionEndpoint,
       },
       async init({ config, logger, workflowLogs }) {
+        if (!config.has('orchestrator.workflowLogProvider.loki')) {
+          logger.info(
+            'Loki workflow log provider is not configured, skipping module initialization',
+          );
+          return;
+        }
         logger.info('Initialize the Loki orchestrator backend module');
         workflowLogs.addWorkflowLogProvider(LokiProvider.fromConfig(config));
       },