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

Alembic migration failing in some edge case #476

Closed
pierrotsmnrd opened this issue Apr 28, 2023 · 1 comment · Fixed by #477
Closed

Alembic migration failing in some edge case #476

pierrotsmnrd opened this issue Apr 28, 2023 · 1 comment · Fixed by #477
Assignees
@pierrotsmnrd
Labels
type: bug 🐛 Something isn't working

Comments

@pierrotsmnrd
Copy link
Contributor

We have identified a case when one alembic migration fails with some edge case data.

Migration with revision 16f65805dc8f fails here in case of data inconsistency.

In the identified case, the reason is dangling build artifacts : rows in table build_artifact with a build_id that doesn't exists anymore.

SELECT * FROM build_artifact
WHERE build_id NOT IN (SELECT DISTINCT(id) FROM build);
/* this returns non-empty result */
@pierrotsmnrd pierrotsmnrd added the type: bug 🐛 Something isn't working label Apr 28, 2023
@pierrotsmnrd pierrotsmnrd self-assigned this Apr 28, 2023
@pierrotsmnrd pierrotsmnrd mentioned this issue Apr 28, 2023
Merged
costrouc pushed a commit that referenced this issue May 2, 2023
@pierrotsmnrd
Fix #476 : delete dangling build artifacts before migration (#477)
ac430b3
@pierrotsmnrd
Copy link
Contributor Author

Related PR : #477 (already merged)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pierrotsmnrd pierrotsmnrd

Labels
type: bug 🐛 Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #476 : delete dangling build artifacts before migration conda-incubator/conda-store
1 participant
@pierrotsmnrd