Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some ❤️ for 🐳 #1238

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Some ❤️ for 🐳 #1238

merged 2 commits into from
Jun 4, 2024

Conversation

sjoerdie
Copy link
Contributor

@sjoerdie sjoerdie commented Jun 4, 2024

  • celery / beat script for docker containers (no activate env)
  • Make docker compose work
  • added CSRF_TRUSTED_ORIGINS, need it to get docker compose working with nginx, without it I get CSRF error when logging on through nginx container. This might break everything.

Docker compose works, including celery task monitoring.
Works on k8s with maykinmedia/charts#110

@sjoerdie sjoerdie requested a review from alextreme June 4, 2024 14:09
Copy link
Member

@alextreme alextreme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zie isort, https://github.com/maykinmedia/open-inwoner/actions/runs/9368671366/job/25791288453?pr=1238

+++ /home/runner/work/open-inwoner/open-inwoner/src/open_inwoner/conf/utils.py:after 2024-06-04 14:12:33.861538
@@ -2,10 +2,10 @@
import os
from shutil import which
from subprocess import CalledProcessError, check_output
+from typing import Any

from django.conf import settings

-from typing import Any
from decouple import Csv, config as _config, undefined
from sentry_sdk.integrations import DidNotEnable, django, redis

@alextreme alextreme merged commit 49de6b4 into develop Jun 4, 2024
18 checks passed
@alextreme alextreme deleted the feature/docker-celery branch June 4, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants