Skip to content

Commit

Permalink
Remove structlog configuration from backend worker
Browse files Browse the repository at this point in the history
Configuring logging for the backend worker requires safir.logging
be installed, which we cannot guarantee is present in the backend
stack container. Remove the configuration for now pending a
discussion about the best way to handle logging from arq.
  • Loading branch information
rra committed Jul 26, 2024
1 parent 5302443 commit 2d2dac2
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions src/vocutouts/workers/cutout.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
WorkerUsageError,
build_worker,
)
from safir.logging import configure_logging
from structlog.stdlib import BoundLogger

from ..models.domain.cutout import (
Expand Down Expand Up @@ -189,12 +188,6 @@ def cutout(
]


configure_logging(
name="vocutouts",
profile=os.getenv("CUTOUT_PROFILE", "development"),
log_level=os.getenv("CUTOUT_LOG_LEVEL", "INFO"),
)

# Provide five seconds of time for arq to shut the worker down cleanly after
# cancelling any running job.
_grace_period = timedelta(seconds=int(os.environ["CUTOUT_GRACE_PERIOD"]))
Expand Down

0 comments on commit 2d2dac2

Please sign in to comment.