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

[stable21] Fix duplicated UUID migration issue #31115

Merged
merged 8 commits into from
Feb 11, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Feb 10, 2022

⚠️ This backport had conflicts and is incomplete ⚠️

Missing is only the reference bump in apps/settings/composer/composer/installed.php 🤷‍♂️

backport of #31109

blizzz and others added 6 commits February 10, 2022 21:40
- in a proper setup there are no duplicated UUIDs
- not all setups are proper
- log warning to admin

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
- to heal incomplete states from broken upgrades

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Generators cannot be iterated with while or returned by an other
 generator, using foreach instead.
And a few other problems.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 21.0.8 milestone Feb 10, 2022
@blizzz blizzz changed the title [stable21] [stable22] Fix duplicated UUID migration issue [stable21] Fix duplicated UUID migration issue Feb 10, 2022
@blizzz blizzz added the 3. to review Waiting for reviews label Feb 10, 2022
Copy link
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

executeQuery

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Copy link
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested with both mysql and postgres, setup check also works

@blizzz
Copy link
Member

blizzz commented Feb 10, 2022

/backport to stable20

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@PVince81 PVince81 merged commit 5380ceb into stable21 Feb 11, 2022
@PVince81 PVince81 deleted the backport/31109/stable21 branch February 11, 2022 09:18
@blizzz blizzz mentioned this pull request Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants