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

updated docker part #9

Closed
wants to merge 2 commits into from
Closed

updated docker part #9

wants to merge 2 commits into from

Conversation

mitjajez
Copy link
Member

Docker build fixed, time to fix django - I need help here

@mitjajez
Copy link
Member Author

I'd also split nginx into it's own separate docker container

@mitar
Copy link
Member

mitar commented May 30, 2020

I would prefer using tozd base image.

@mitjajez
Copy link
Member Author

So https://github.com/tozd/docker-postgresql and https://github.com/tozd/docker-nginx
I can try with those ..
What about updating those docker images since they are not official?

@mitar
Copy link
Member

mitar commented May 30, 2020

I think we should first get this to run with old versions. And then we can start updating them to the latest versions. I was also planing to update those images anyway, so I can look into that.

@mitjajez
Copy link
Member Author

mitjajez commented Jun 2, 2020

docker-compose up return

app_1  | bash: cannot set terminal process group (-1): Inappropriate ioctl for device
app_1  | bash: no job control in this shell

it continue
but ended in scripts/migrate.py#L13
python manage.py syncdb --noinput
with error:
django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet

How to get out witch app isn't loaded yet? Should i go over INSTALLED_APPS in settings.py uncommenting one by one?

@rasovica
Copy link

rasovica commented Jun 2, 2020

Look at my pr #7 I think I got a few steps further, I think I got stuck with making nodewatcher run as it used some incompatible / broken deps

@mitjajez
Copy link
Member Author

mitjajez commented Jun 2, 2020

I was not able to build your pr, but I might put some deps too new. I'm now downgrading django 1.11 (latest with python2) -> django 1.7
Or should I restart from your pr?

@rasovica
Copy link

rasovica commented Jun 2, 2020

Its probably best for you to make your own, I am just saying I was going through the same pains and I feel like I got a bit further than you. Maybe you can find some problems already solved on my branch.

@mitjajez
Copy link
Member Author

mitjajez commented Jun 2, 2020

oh, now remember: main problem I found with #7 is db image.

Step 3/14 : ADD pgdg.list /etc/apt/sources.list.d/pgdg.list
ERROR: Service 'db' failed to build: ADD failed: stat /var/lib/docker/tmp/docker-builder251930779/pgdg.list: no such file or directory

That's why I'm trying with image: tozd/postgresql:9.6 in docker-compose.yml line

@mitjajez
Copy link
Member Author

mitjajez commented Jun 2, 2020

I will open new PR with less changes from feature/revive.

@mitjajez mitjajez closed this Jun 2, 2020
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.

3 participants