Skip to content

feat: move to postgresql#4744

Merged
rjsparks merged 7 commits into
ietf-tools:feat/postgresfrom
rjsparks:postgres
Nov 11, 2022
Merged

feat: move to postgresql#4744
rjsparks merged 7 commits into
ietf-tools:feat/postgresfrom
rjsparks:postgres

Conversation

@rjsparks

Copy link
Copy Markdown
Member

No description provided.

Comment thread docker/scripts/db-include-fix.py Outdated
@jennifer-richards
jennifer-richards self-requested a review November 10, 2022 19:34

@jennifer-richards jennifer-richards left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested a tweak to the db-include-fix.py that gets it working on my system. Might also be worth adding some debug output to indicate whether that script makes changes or not - it's a little fragile. OTOH may not have a long enough lifespan to warrant it.

The pause migration is still not working as intended, haven't sorted out why.

Comment thread docker/scripts/db-include-fix.py Outdated
Comment thread docker/scripts/db-include-fix.py Outdated
Comment thread docker/scripts/db-include-fix.py Outdated
Comment thread ietf/utils/migrations/0004_pause_to_change_database_engines.py Outdated
Comment thread ietf/utils/migrations/0004_pause_to_change_database_engines.py Outdated
rjsparks and others added 5 commits November 11, 2022 10:04
Co-authored-by: Jennifer Richards <jennifer@painless-security.com>
…ows away the first.

Co-authored-by: Jennifer Richards <jennifer@painless-security.com>
Co-authored-by: Jennifer Richards <jennifer@painless-security.com>
Co-authored-by: Jennifer Richards <jennifer@painless-security.com>
Co-authored-by: Jennifer Richards <jennifer@painless-security.com>
@rjsparks
rjsparks merged commit 27fccc6 into ietf-tools:feat/postgres Nov 11, 2022
@rjsparks
rjsparks deleted the postgres branch November 11, 2022 11:03
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants