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

chore(db): Améliorer l'hydratation de la base de données en developpement #777

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,13 @@ index:
INSTANCE_NUMBER=0 \
POSTGRESQL_ADDON_URI=$(POSTGRESQL_ADDON_URI) \
clevercloud/rebuild_index.sh

# DB

.PHONY: resetdb
resetdb:
dropdb --if-exists $(POSTGRESQL_ADDON_DB)
createdb $(POSTGRESQL_ADDON_DB)
python manage.py migrate
python manage.py configure_bucket
Copy link
Contributor

Choose a reason for hiding this comment

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

Pinaillage, mais ce n’est pas de la DB, c’est pour minio.
Une fois que c’est fait, normalement pas besoin de le refaire.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

💯

python manage.py populate
10 changes: 5 additions & 5 deletions scripts/import-latest-db-backup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,16 +23,16 @@ echo "Going to inject DB_BACKUP_PATH=$DB_BACKUP_PATH"
docker cp $DB_BACKUP_PATH commu_postgres:/backups

echo "dropping current db"
docker exec -ti commu_postgres dropdb $POSTGRESQL_ADDON_DB -U $POSTGRESQL_ADDON_USER --if-exists --echo
dropdb $POSTGRESQL_ADDON_DB -U $POSTGRESQL_ADDON_USER --if-exists --echo

echo "creating new db"
docker exec -ti commu_postgres createdb $POSTGRESQL_ADDON_DB -O $POSTGRESQL_ADDON_USER -U $POSTGRESQL_ADDON_USER --echo
createdb $POSTGRESQL_ADDON_DB -O $POSTGRESQL_ADDON_USER -U $POSTGRESQL_ADDON_USER --echo

echo "restoring db"
docker exec -ti commu_postgres pg_restore -U $POSTGRESQL_ADDON_USER --dbname=$POSTGRESQL_ADDON_DB --format=c --clean --no-owner --verbose /backups/$DB_BACKUP_NAME
pg_restore -U $POSTGRESQL_ADDON_USER --dbname=$POSTGRESQL_ADDON_DB --format=c --clean --no-owner $DB_BACKUP_PATH

echo "restarting db"
docker-compose down postgres; docker-compose up postgres -d
Comment on lines -26 to -35
Copy link
Contributor

Choose a reason for hiding this comment

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

Ces changements font que le dev doit avoir installé le client PostgreSQL sur sa machine. Pas bien grave, mais peut-être l’ajouter dans le README ?

echo "applying new migrations"
python manage.py migrate

echo "faking password"
python manage.py set_fake_passwords
Expand Down
Loading