vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity
This repository has been archived on 2020-03-24. You can view files and clone it, but cannot push or open issues or pull requests.
quay/data/migrations/versions/5b7503aada1b_cleanup_old_robots.py

24 lines
438 B
Python
Raw Normal View History

Fix issue where we didn't delete robots immediately under a namespace This could result in "hanging" robot accounts, although that would only leak the names of said accounts. Now we delete them immediately AND we proactively delete them before replacing the namespace (just to be sure)
2018-05-09 11:52:28 +00:00
"""Cleanup old robots
Revision ID: 5b7503aada1b
Revises: 224ce4c72c2f
Create Date: 2018-05-09 17:18:52.230504
"""
# revision identifiers, used by Alembic.
revision = '5b7503aada1b'
down_revision = '224ce4c72c2f'
from alembic import op
import sqlalchemy as sa
from util.migrate.cleanup_old_robots import cleanup_old_robots
Add support for populating test data during migration testing This change ensures that the tables in the database during migration have at least one row of "real" data, which should help catch issues in the future where we forget to set column defaults and other such schema oversights that can only be truly tested with non-empty tables Fixes https://jira.coreos.com/browse/QUAY-913
2018-05-07 13:45:57 +00:00
def upgrade(tables, tester):
Fix issue where we didn't delete robots immediately under a namespace This could result in "hanging" robot accounts, although that would only leak the names of said accounts. Now we delete them immediately AND we proactively delete them before replacing the namespace (just to be sure)
2018-05-09 11:52:28 +00:00
cleanup_old_robots()
Add support for populating test data during migration testing This change ensures that the tables in the database during migration have at least one row of "real" data, which should help catch issues in the future where we forget to set column defaults and other such schema oversights that can only be truly tested with non-empty tables Fixes https://jira.coreos.com/browse/QUAY-913
2018-05-07 13:45:57 +00:00
def downgrade(tables, tester):
Fix issue where we didn't delete robots immediately under a namespace This could result in "hanging" robot accounts, although that would only leak the names of said accounts. Now we delete them immediately AND we proactively delete them before replacing the namespace (just to be sure)
2018-05-09 11:52:28 +00:00
# Nothing to do.
pass
Reference in a new issue Copy permalink