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 missing table name #1817

Merged
merged 2 commits into from
Jun 3, 2023
Merged

Fix missing table name #1817

merged 2 commits into from
Jun 3, 2023

Conversation

qwerty287
Copy link
Contributor

@qwerty287 qwerty287 commented Jun 3, 2023

Bug from #1806

We don't need the sync because the step_machine is there since the model was added the first time so it mill always be in the table

@qwerty287 qwerty287 added bug Something isn't working regression fix a bug that was not released yet labels Jun 3, 2023
@qwerty287 qwerty287 added this to the 1.0.0 milestone Jun 3, 2023
@qwerty287 qwerty287 requested a review from a team June 3, 2023 16:24
@6543 6543 merged commit b9731d8 into woodpecker-ci:master Jun 3, 2023
@pat-s pat-s mentioned this pull request Jun 4, 2023
5 tasks
qwerty287 added a commit that referenced this pull request Jun 4, 2023
6543 added a commit that referenced this pull request Jun 6, 2023
…ght away (#1822)

This isolates single migration tasks from each other.
The migration itself is now not atomic anymore but each single migration
now on it's own.
This takes load away from databases, as new sessions have a committed
schema available.

We also disable xorm.cache, as the speed improvements are minor but
invalid cache caused by schema changes did happen already in the past.

---------
Reverts #1817
Closes #1821

---------

Co-authored-by: 6543 <6543@obermui.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression fix a bug that was not released yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants