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.

Sign up for GitHub

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

Jump to bottom

Fix bad drop_constraint call in migrations #31302

Merged
merged 1 commit into from
May 15, 2023
Merged

Fix bad drop_constraint call in migrations #31302

merged 1 commit into from
May 15, 2023

Conversation

dstandish
Copy link
Contributor

Apparently alembic no longer likes 2 positional arguments.

@dstandish
Fix bad drop_constraint call in migrations
17539c0 
Apparently alembic no longer likes 2 positional arguments.

(cherry picked from commit dba5ec3)
@dstandish dstandish requested a review from potiuk as a code owner May 15, 2023 22:27
@jedcunningham jedcunningham added this to the Airflow 2.6.2 milestone May 15, 2023
@jedcunningham jedcunningham merged commit 1bd538b into apache:main May 15, 2023
@jedcunningham jedcunningham deleted the fix-alembic-drop-constraint-issue branch May 15, 2023 23:03
@jedcunningham jedcunningham added the type:bug-fix Changelog: Bug Fixes label May 15, 2023
@potiuk
Copy link
Member

potiuk commented May 16, 2023

Not nice, alembic, not nice :)

potiuk added a commit to potiuk/airflow that referenced this pull request May 16, 2023
@potiuk
Use keyword paramaters for migration methods for mssql
ea1220d 
The new alembic (1.11.0) requires parameters to be keyword
parameters in methods used in migrations. Part of the problem has
been fixed in apache#31306 and apache#31302 but still some migrations that are
specifically run in case of mssql need to use migration parameters.
@potiuk potiuk mentioned this pull request May 16, 2023
Merged
potiuk added a commit that referenced this pull request May 16, 2023
@potiuk
Use keyword paramaters for migration methods for mssql (#31309)
5479027 
The new alembic (1.11.0) requires parameters to be keyword
parameters in methods used in migrations. Part of the problem has
been fixed in #31306 and #31302 but still some migrations that are
specifically run in case of mssql need to use migration parameters.
@potiuk potiuk mentioned this pull request May 16, 2023
Closed
2 tasks
eladkal pushed a commit that referenced this pull request Jun 8, 2023
@potiuk
Use keyword paramaters for migration methods for mssql (#31309)
12f17cc 
The new alembic (1.11.0) requires parameters to be keyword
parameters in methods used in migrations. Part of the problem has
been fixed in #31306 and #31302 but still some migrations that are
specifically run in case of mssql need to use migration parameters.

(cherry picked from commit 5479027)
potiuk added a commit that referenced this pull request Jun 9, 2023
@potiuk
Use keyword paramaters for migration methods for mssql (#31309)
ea8c0a1 
The new alembic (1.11.0) requires parameters to be keyword
parameters in methods used in migrations. Part of the problem has
been fixed in #31306 and #31302 but still some migrations that are
specifically run in case of mssql need to use migration parameters.

(cherry picked from commit 5479027)
@eladkal eladkal mentioned this pull request Jun 13, 2023
Closed
62 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincbeck vincbeck vincbeck approved these changes

@jedcunningham jedcunningham jedcunningham approved these changes

@potiuk potiuk Awaiting requested review from potiuk potiuk is a code owner

Assignees
No one assigned
Labels
kind:documentation type:bug-fix Changelog: Bug Fixes
Projects
None yet
Milestone
Airflow 2.6.2
Development

Successfully merging this pull request may close these issues.
4 participants
@dstandish @potiuk @jedcunningham @vincbeck