Skip to content

[stable17] Delete chunks if the move on an upload failed #22241

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
Merged
merged 3 commits into from
Aug 19, 2020
Merged

[stable17] Delete chunks if the move on an upload failed #22241

merged 3 commits into from
Aug 19, 2020

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Aug 13, 2020 •
edited by nickvergessen
Loading

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #22128

@juliusknorr @backportbot
Delete chunks if the move on an upload failed
3802330 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr @backportbot
Adjust chunking test for non-existing target node
e9bd87a 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 17.0.10 milestone Aug 13, 2020
@rullzer rullzer added the 3. to review Waiting for reviews label Aug 14, 2020
@rullzer rullzer mentioned this pull request Aug 18, 2020
Merged
@nickvergessen nickvergessen self-assigned this Aug 19, 2020
@juliusknorr @nickvergessen
Return proper status when file didn't exist before
4bc8601 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@nickvergessen nickvergessen force-pushed the backport/22128/stable17 branch from 5541325 to 4bc8601 Compare August 19, 2020 13:38
@nickvergessen
Copy link
Member

Updated the broken patch, works now

@rullzer rullzer added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 19, 2020
@faily-bot
Copy link

faily-bot bot commented Aug 19, 2020

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 31972: failure

webpack-build

mariadb10.4-php7.3

  • cancelled - typically means that the tests took longer than the drone CI allows them to run

@MorrisJobke MorrisJobke merged commit 61328bd into stable17 Aug 19, 2020
@MorrisJobke MorrisJobke deleted the backport/22128/stable17 branch August 19, 2020 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rullzer rullzer rullzer approved these changes

@nickvergessen nickvergessen nickvergessen approved these changes

@blizzz blizzz Awaiting requested review from blizzz

@georgehrke georgehrke Awaiting requested review from georgehrke

@icewind1991 icewind1991 Awaiting requested review from icewind1991

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@MorrisJobke MorrisJobke Awaiting requested review from MorrisJobke

Assignees

@nickvergessen nickvergessen

Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Milestone
Nextcloud 17.0.9
Development

Successfully merging this pull request may close these issues.
4 participants
@nickvergessen @rullzer @MorrisJobke @juliusknorr