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

installing postgresql twice #23

Closed
ltalirz opened this issue Sep 30, 2019 · 2 comments
Closed

installing postgresql twice #23

ltalirz opened this issue Sep 30, 2019 · 2 comments

Comments

@ltalirz
Copy link
Member

ltalirz commented Sep 30, 2019

@yakutovicha You're currently installing postgresql twice in the aiidalab-1.0-migration branch:

version 10

postgresql \

and version 9.6

# Add repo for postgres-9.6
RUN echo "deb http://apt.postgresql.org/pub/repos/apt/ bionic-pgdg main" >> /etc/apt/sources.list.d/pgdg.list
RUN wget -q https://www.postgresql.org/media/keys/ACCC4CF8.asc -O - | apt-key add -
RUN apt-get update && apt-get install -y --no-install-recommends \
postgresql-9.6 \
&& rm -rf /var/lib/apt/lists/*

What ends up running is version 10:

# ps -ef | grep postgresql
scienti+    43     1  0 19:59 ?        00:00:00 /usr/lib/postgresql/10/bin/postgres -D /project/.postgresql

This will bite you when you try to migrate databases from the previous image that was running postgres 9.6.
Either you add some migration scripts or you switch back to version 9.6

@yakutovicha
Copy link
Member

I choose to run version 10 and provide migration scripts. This is anyways done for aiida-1.0 (0.12 uses Postgres 9.6)

Thanks for catching this error!

@yakutovicha
Copy link
Member

This is now fixed, I created a separate issue #25 to remind myself about migration scripts.

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

No branches or pull requests

2 participants