Skip to content

Commit a6933b2

Browse files
make worker logs on database optional
1 parent 03ad47b commit a6933b2

File tree

1 file changed

+17
-12
lines changed

1 file changed

+17
-12
lines changed

cads_worker/worker.py

+17-12
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import datetime
2+
import distutils.util
23
import functools
34
import os
45
import random
@@ -18,6 +19,8 @@
1819

1920
LOGGER = structlog.get_logger(__name__)
2021

22+
WORKER_LOGS_ON_DB = distutils.util.strtobool(os.getenv("WORKER_LOGS_ON_DB", "true"))
23+
2124

2225
@functools.lru_cache
2326
def create_session_maker() -> cads_broker.database.sa.orm.sessionmaker:
@@ -106,12 +109,13 @@ def add_stdout(
106109
self.logger.warning(message, job_id=job_id)
107110
if log_type == "critical":
108111
self.logger.critical(message, job_id=job_id)
109-
cads_broker.database.add_event(
110-
event_type=log_type,
111-
request_uid=job_id,
112-
message=message,
113-
session=session,
114-
)
112+
if WORKER_LOGS_ON_DB:
113+
cads_broker.database.add_event(
114+
event_type=log_type,
115+
request_uid=job_id,
116+
message=message,
117+
session=session,
118+
)
115119

116120
@ensure_session
117121
def add_stderr(
@@ -128,12 +132,13 @@ def add_stderr(
128132
self.logger.exception(message, job_id=job_id)
129133
if log_type == "error":
130134
self.logger.error(message, job_id=job_id)
131-
cads_broker.database.add_event(
132-
event_type=log_type,
133-
request_uid=job_id,
134-
message=message,
135-
session=session,
136-
)
135+
if WORKER_LOGS_ON_DB:
136+
cads_broker.database.add_event(
137+
event_type=log_type,
138+
request_uid=job_id,
139+
message=message,
140+
session=session,
141+
)
137142

138143
@property
139144
def session_maker(self) -> cads_broker.database.sa.orm.sessionmaker:

0 commit comments

Comments
 (0)