worker: metrics server startup if correct env var set

see PR 85 for context. Also removed double guard of same setting: worker_settings.metrics.enabled

Author: RMCrean

packages/divbase-api/src/divbase_api/worker/metrics.py

@@ -321,9 +321,6 @@ def start_metrics_server(port=8101):
     For k8s deployment, concurrency must be set to 1 (1 worker process per pod). Scaling will be handled by increasing the number of pods.
     Example with concurrency=1: 1 worker process, 1 collection thread, 1 HTTP server.
     """
-    if not worker_settings.metrics.enabled:
-        logger.info("Metrics collection disabled via ENABLE_WORKER_METRICS environment variable")
-        return
     global _metrics_server_started
 
     pid = os.getpid()

packages/divbase-api/src/divbase_api/worker/tasks.py

@@ -99,11 +99,11 @@ def init_worker_metrics(**kwargs):
     Start the Prometheus metrics server for the Celery worker. This signal is triggered once per forked worker process.
     With celery prefork concurrency=1, only one worker process will start the metrics server.
     """
-    if worker_settings.metrics.enabled_per_task:
-        logger.info("Starting worker metrics server on port 8101 (ENABLE_WORKER_METRICS_PER_TASK is set to '1').")
+    if worker_settings.metrics.enabled:
+        logger.info("Starting worker metrics server on port 8101.")
         start_metrics_server(port=8101)
     else:
-        logger.info("Worker metrics collection disabled (ENABLE_WORKER_METRICS_PER_TASK not set to '1').")
+        logger.info("Worker metrics collection disabled (ENABLE_WORKER_METRICS not set to '1').")
 
 
 def dynamic_router(name, args, kwargs, options, task=None, **kw):