Skip to content

Fix size calculation on copying the skeleton files - #34834

Merged
PVince81 merged 1 commit into
masterfrom
bugfix/noid/skeleton-size-calc
Nov 4, 2022
Merged

Fix size calculation on copying the skeleton files#34834
PVince81 merged 1 commit into
masterfrom
bugfix/noid/skeleton-size-calc

Conversation

@juliusknorr

@juliusknorrjuliusknorr commented Oct 27, 2022

Copy link
Copy Markdown
Member

Use proper storage methods to copy the skeleton files as otherwise the size propagation for the newly added files (and the user quota usage) will be wrong after the first login with object storage.

For local storage this works fine as the files are getting scanned afterwards.

otherwise the filecache will have a wrong size for skeleton files
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr
juliusknorr requested review from a team, CarlSchwan, come-nc and icewind1991 and removed request for a teamOctober 27, 2022 06:49
@juliusknorrjuliusknorr added bug 3. to review Waiting for reviews labels Oct 27, 2022
@juliusknorrjuliusknorr added this to the Nextcloud 26 milestone Oct 27, 2022
@juliusknorr
juliusknorr requested a review from a teamOctober 31, 2022 16:04

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@PVince81
PVince81 merged commit 966ee67 into masterNov 4, 2022
@PVince81
PVince81 deleted the bugfix/noid/skeleton-size-calc branch November 4, 2022 16:23
@PVince81

Copy link
Copy Markdown
Member

backports needed ?

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Yes, would be good.

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable25

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable24

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable24 failed. Please do this backport manually.

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable25 failed. Please do this backport manually.

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable23 failed. Please do this backport manually.

@nickvergessen

Copy link
Copy Markdown
Member

Brings a strange regression on 24 + 23
#35049 (comment)

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

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@juliusknorr@PVince81@nickvergessen@come-nc