-
Notifications
You must be signed in to change notification settings - Fork 987
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
Fixes #33660 - ignore dynflow tables for dump #8822
Fixes #33660 - ignore dynflow tables for dump #8822
Conversation
Issues: #33660 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected, no mentions of dynflow in db/schema.rb anymore
Even though it seems to work on my machine, test failures seem related |
@lzap ping |
8f446f7
to
140a224
Compare
Rebased. |
Signed-off-by: Lukas Zapletal <lzap+git@redhat.com>
140a224
to
7e588ce
Compare
Rebased! |
Seems to work well, waiting for the tests to pass |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests were fixed in another pr, let's get this in
Thank you @lzap ! |
Ruby schema dump contains dynflow tables again, which is causing issues with foreign keys with multi-column PKs.
Regression introduced in #8660