Skip to content

Conversation

@icewind1991
Copy link
Member

Changes the object store background scan from a noop to repairing -1 sizes properly.

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Apr 12, 2023
@icewind1991 icewind1991 added this to the Nextcloud 27 milestone Apr 12, 2023
@icewind1991 icewind1991 requested review from a team, ArtificialOwl, blizzz and come-nc and removed request for a team April 12, 2023 16:09
@icewind1991 icewind1991 force-pushed the object-store-background-scan branch from 8a4e275 to d101999 Compare April 28, 2023 13:33
@icewind1991 icewind1991 requested a review from Pytal May 2, 2023 13:19
@Pytal Pytal requested a review from susnux May 3, 2023 00:35
@skjnldsv skjnldsv mentioned this pull request May 3, 2023
Comment on lines +92 to +64
// FIXME: this won't proceed with the next item, needs revamping of getIncomplete()
// to make this possible
Copy link
Member

Choose a reason for hiding this comment

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

when? ;)

Copy link
Member Author

Choose a reason for hiding this comment

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

eventually™.

it would require some deeper rework of how unscanned files are detected which I wouldn't want to do here

This was referenced May 9, 2023
@skjnldsv skjnldsv added the bug label May 9, 2023
@icewind1991 icewind1991 force-pushed the object-store-background-scan branch from d101999 to b1327f1 Compare May 9, 2023 15:31
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 9, 2023
@icewind1991 icewind1991 force-pushed the object-store-background-scan branch from b1327f1 to ff8feca Compare May 10, 2023 11:21
@icewind1991 icewind1991 enabled auto-merge May 10, 2023 15:23
@icewind1991 icewind1991 force-pushed the object-store-background-scan branch from ff8feca to 2ea41da Compare May 10, 2023 17:33
@skjnldsv skjnldsv disabled auto-merge May 11, 2023 06:15
@skjnldsv skjnldsv merged commit db02684 into master May 11, 2023
@skjnldsv skjnldsv deleted the object-store-background-scan branch May 11, 2023 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants