Skip to content

Commit c6dfbb2

Browse files
committed
change to json-formatted logging
1 parent c2dfd13 commit c6dfbb2

File tree

4 files changed

+13
-100
lines changed

4 files changed

+13
-100
lines changed

Dockerfile

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,8 @@ ENV LANG=en_US.UTF-8 \
121121
VIRTUAL_ENV=/opt/invenio/.venv \
122122
PATH="/opt/invenio/.venv/bin:$PATH" \
123123
WORKING_DIR=/opt/invenio \
124-
INVENIO_INSTANCE_PATH=/opt/invenio/var/instance
124+
INVENIO_INSTANCE_PATH=/opt/invenio/var/instance \
125+
GUNICORN_ACCESS_LOGFORMAT="{\"time\":\"%(t)s\",\"ip\":\"%({x-forwarded-for}i)s\",\"method\":\"%(m)s\",\"path\":\"%(U)s\",\"query\":\"%(q)s\",\"status\":%(s)s,\"size\":%(B)s,\"referer\":\"%(f)s\",\"user_agent\":\"%(a)s\",\"request_time\":%(L)s}"
125126

126127
# create non-root invenio user
127128
ENV INVENIO_USER_ID=1654
@@ -150,11 +151,9 @@ COPY --from=builder --chown=1654:0 ${INVENIO_INSTANCE_PATH}/invenio.cfg ${INVENI
150151
COPY --from=builder --chown=1654:0 --chmod=755 ${INVENIO_INSTANCE_PATH}/update_subjects.py ${INVENIO_INSTANCE_PATH}/update_subjects.py
151152
COPY --chown=1654:0 ./Caddyfile /etc/caddy/Caddyfile
152153
COPY --chown=1654:0 --chmod=755 ./entrypoint.sh /opt/invenio/.venv/bin/entrypoint.sh
153-
COPY --chown=1654:0 ./entrypoint.py /opt/invenio/.venv/bin/entrypoint.py
154154

155155
WORKDIR ${WORKING_DIR}/src
156156

157157
USER invenio
158158
EXPOSE 4000
159-
# ENTRYPOINT ["python", "/opt/invenio/.venv/bin/entrypoint.py"]
160-
CMD ["sh", "-c", "gunicorn invenio_app.wsgi:application --bind 0.0.0.0:4000 --workers 2 --threads 4 --timeout 60 --access-logfile - --error-logfile - --log-level ${GUNICORN_LOG_LEVEL:-WARNING}"]
159+
CMD ["sh", "-c", "gunicorn invenio_app.wsgi:application --bind 0.0.0.0:4000 --workers 2 --threads 4 --timeout 60 --access-logfile - --access-logformat \"${GUNICORN_ACCESS_LOGFORMAT}\" --error-logfile - --log-level ${GUNICORN_LOG_LEVEL:-WARNING}"]
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ x-common-env:
3737
INVENIO_LOGGING_CONSOLE_LEVEL: ${INVENIO_LOG_LEVEL:-WARNING}
3838
INVENIO_LOGGING_FS_LEVEL: ${INVENIO_LOG_LEVEL:-WARNING}
3939
GUNICORN_LOG_LEVEL: ${INVENIO_LOG_LEVEL:-WARNING}
40+
INVENIO_CELERY_WORKER_LOG_FORMAT: ${INVENIO_CELERY_WORKER_LOG_FORMAT:-{"time":"%(asctime)s","level":"%(levelname)s","logger":"%(name)s","message":"%(message)s"}}
41+
INVENIO_CELERY_WORKER_TASK_LOG_FORMAT: ${INVENIO_CELERY_WORKER_TASK_LOG_FORMAT:-{"time":"%(asctime)s","level":"%(levelname)s","logger":"%(name)s","task_name":"%(task_name)s","task_id":"%(task_id)s","message":"%(message)s"}}
4042

4143
# DOI Registration
4244
INVENIO_DATACITE_ENABLED: ${INVENIO_DATACITE_ENABLED:-False}

entrypoint.py

Lines changed: 0 additions & 96 deletions
This file was deleted.

invenio.cfg

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,14 @@ ACCOUNTS_SESSION_REDIS_URL = _redis_url
179179
CELERY_BROKER_URL = f"{_redis_base}/{_redis_db + 1}"
180180
BROKER_URL = CELERY_BROKER_URL
181181
CELERY_RESULT_BACKEND = f"{_redis_base}/{_redis_db + 2}"
182+
CELERY_WORKER_LOG_FORMAT = os.environ.get(
183+
"INVENIO_CELERY_WORKER_LOG_FORMAT",
184+
'{"time":"%(asctime)s","level":"%(levelname)s","logger":"%(name)s","message":"%(message)s"}',
185+
)
186+
CELERY_WORKER_TASK_LOG_FORMAT = os.environ.get(
187+
"INVENIO_CELERY_WORKER_TASK_LOG_FORMAT",
188+
'{"time":"%(asctime)s","level":"%(levelname)s","logger":"%(name)s","task_name":"%(task_name)s","task_id":"%(task_id)s","message":"%(message)s"}',
189+
)
182190
RATELIMIT_STORAGE_URL = f"{_redis_base}/{_redis_db + 3}"
183191
CACHE_REDIS_URL = f"{_redis_base}/{_redis_db + 4}"
184192
COMMUNITIES_IDENTITIES_CACHE_REDIS_URL = CACHE_REDIS_URL

0 commit comments

Comments
 (0)