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

Fix db downgrades #19994

Merged
merged 1 commit into from
Dec 3, 2021
Merged

Fix db downgrades #19994

merged 1 commit into from
Dec 3, 2021

Conversation

jedcunningham
Copy link
Member

Downgrading from 2.2.0 wasn't working on Postgres or MySQL.

Closes #19945

@github-actions
Copy link

github-actions bot commented Dec 2, 2021

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Dec 2, 2021
@potiuk
Copy link
Member

potiuk commented Dec 3, 2021

Needs rebase for another migration?

@jedcunningham
Copy link
Member Author

Needs rebase for another migration?

Not really, it was just 1 commit behind, but I've rebased again to get that extra green check mark 👍

@potiuk
Copy link
Member

potiuk commented Dec 3, 2021

That's a protection to avoid accidental "double-head" case BTW

Downgrading from 2.2.0 wasn't working on Postgres or MySQL.
@jedcunningham jedcunningham merged commit 14bff0e into apache:main Dec 3, 2021
@jedcunningham jedcunningham deleted the fix_downgrades branch December 3, 2021 20:36
@jedcunningham jedcunningham added this to the Airflow 2.2.3 milestone Dec 7, 2021
jedcunningham added a commit that referenced this pull request Dec 7, 2021
Downgrading from 2.2.0 wasn't working on Postgres or MySQL.

(cherry picked from commit 14bff0e)
@jedcunningham jedcunningham added the type:bug-fix Changelog: Bug Fixes label Dec 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full tests needed We need to run full set of tests for this PR to merge type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Running alembic migration downgrade on database causes error
3 participants