Change Details

For the purposes of moving to Postgres, Ricki refactored the docker compose setup to use an entrypoint script, which upgraded the database with all existing revisions, migrated data from the old SQLite data file, checked the Great Expectations assumptions about the data, and then started the server. We wish to remove the migration step, as it has now successfully migrated data and does not need to re-run on each startup. (Not least, it takes 15+ minutes to run each time, adding considerably to startup times.) AC: [] Migration step removed from backend/entrypoint.sh [] $DATABASE_URL_OLD environment variable removed from docker-compose.yml Note: Please leave migration script, backend/migrate.py, intact, just in case we need to migrate data again in the future. Removal of old database string from config is left to the implementer's discretion.

For the purposes of moving to Postgres, Ricki refactored the docker compose setup to use an entrypoint script, which upgraded the database with all existing revisions, migrated data from the old SQLite data file, checked the Great Expectations assumptions about the data, and then started the server. We wish to remove the migration step, as it has now successfully migrated data and does not need to re-run on each startup. (Not least, it takes 15+ minutes to run each time, adding considerably to startup times.) AC: [] Migration step removed from backend/entrypoint.sh [] $DATABASE_URL_OLD environment variable removed from docker-compose.yml [] backend/migrate.py added to .dockerignore Note: Please leave migration script, backend/migrate.py, intact, just in case we need to migrate data again in the future. Removal of old database string from config is left to the implementer's discretion.

For the purposes of moving to Postgres, Ricki refactored the docker compose setup to use an entrypoint script, which upgraded the database with all existing revisions, migrated data from the old SQLite data file, checked the Great Expectations assumptions about the data, and then started the server. We wish to remove the migration step, as it has now successfully migrated data and does not need to re-run on each startup. (Not least, it takes 15+ minutes to run each time, adding considerably to startup times.) AC: [] Migration step removed from backend/entrypoint.sh [] $DATABASE_URL_OLD environment variable removed from docker-compose.yml [] backend/migrate.py added to .dockerignore Note: Please leave migration script, backend/migrate.py, intact, just in case we need to migrate data again in the future. Removal of old database string from config is left to the implementer's discretion.