Skip to content

Conversation

@provokateurin
Copy link
Member

Some object stores might bill you more once the size is over a certain limits, so admins might want to prevent that case.

@provokateurin provokateurin added this to the Nextcloud 33 milestone Sep 2, 2025
@provokateurin provokateurin requested a review from a team as a code owner September 2, 2025 09:22
@provokateurin provokateurin requested review from sorbaugh and removed request for a team September 2, 2025 09:22
@provokateurin provokateurin force-pushed the object-store-total-size-limit branch 2 times, most recently from 81a91a6 to 1df09c1 Compare September 8, 2025 12:04
@provokateurin

This comment was marked as resolved.

@provokateurin
Copy link
Member Author

Fixed the size propagation in #54950

Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

I was expecting to feed this into IStorage::free_space instead of adding our own check.
But I’m not sure whether that would work as expected?

@provokateurin
Copy link
Member Author

Oh that's also a good idea, I didn't consider that 😅
I'll take a look at it, it probably makes more sense to put it there, as the logic around it should already be capable of handling the limits correctly.

@provokateurin provokateurin force-pushed the object-store-total-size-limit branch from 1df09c1 to 98081df Compare September 10, 2025 06:15
@github-actions
Copy link
Contributor

Possible performance regression detected

Show Output
An unhandled exception has been thrown:
TypeError: array_map(): Argument #2 ($array) must be of type array, null given in /home/runner/work/server/server/apps/profiler/lib/Command/Compare.php:35
Stack trace:
#0 /home/runner/work/server/server/apps/profiler/lib/Command/Compare.php(35): array_map()
#1 /home/runner/work/server/server/3rdparty/symfony/console/Command/Command.php(326): OCAProfilerCommandCompare->execute()
#2 /home/runner/work/server/server/core/Command/Base.php(218): SymfonyComponentConsoleCommandCommand->run()
#3 /home/runner/work/server/server/3rdparty/symfony/console/Application.php(1078): OCCoreCommandBase->run()
#4 /home/runner/work/server/server/3rdparty/symfony/console/Application.php(324): SymfonyComponentConsoleApplication->doRunCommand()
#5 /home/runner/work/server/server/3rdparty/symfony/console/Application.php(175): SymfonyComponentConsoleApplication->doRun()
#6 /home/runner/work/server/server/lib/private/Console/Application.php(187): SymfonyComponentConsoleApplication->run()
#7 /home/runner/work/server/server/console.php(91): OCConsoleApplication->run()
#8 /home/runner/work/server/server/occ(33): require_once('...')
#9 {main}

@github-actions
Copy link
Contributor

Possible performance regression detected

Show Output
An unhandled exception has been thrown:
TypeError: array_map(): Argument #2 ($array) must be of type array, null given in /home/runner/actions-runner/_work/server/server/apps/profiler/lib/Command/Compare.php:35
Stack trace:
#0 /home/runner/actions-runner/_work/server/server/apps/profiler/lib/Command/Compare.php(35): array_map()
#1 /home/runner/actions-runner/_work/server/server/3rdparty/symfony/console/Command/Command.php(326): OCAProfilerCommandCompare->execute()
#2 /home/runner/actions-runner/_work/server/server/core/Command/Base.php(218): SymfonyComponentConsoleCommandCommand->run()
#3 /home/runner/actions-runner/_work/server/server/3rdparty/symfony/console/Application.php(1078): OCCoreCommandBase->run()
#4 /home/runner/actions-runner/_work/server/server/3rdparty/symfony/console/Application.php(324): SymfonyComponentConsoleApplication->doRunCommand()
#5 /home/runner/actions-runner/_work/server/server/3rdparty/symfony/console/Application.php(175): SymfonyComponentConsoleApplication->doRun()
#6 /home/runner/actions-runner/_work/server/server/lib/private/Console/Application.php(187): SymfonyComponentConsoleApplication->run()
#7 /home/runner/actions-runner/_work/server/server/console.php(91): OCConsoleApplication->run()
#8 /home/runner/actions-runner/_work/server/server/occ(33): require_once('...')
#9 {main}

Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

Clean.
Were you able to test it?

@provokateurin
Copy link
Member Author

Were you able to test it?

Yes, tested with some simple limits.

@come-nc come-nc requested a review from artonge September 26, 2025 10:06
Copy link
Member

@AndyScherzinger AndyScherzinger left a comment

Choose a reason for hiding this comment

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

🐘

@provokateurin provokateurin merged commit c5c3b65 into master Sep 27, 2025
213 of 220 checks passed
@provokateurin provokateurin deleted the object-store-total-size-limit branch September 27, 2025 13:02
@skjnldsv skjnldsv removed this from the Nextcloud 33 milestone Oct 2, 2025
@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
@come-nc come-nc added the pending documentation This pull request needs an associated documentation update label Oct 7, 2025
@come-nc come-nc modified the milestones: Nextcloud 32, Nextcloud 33 Oct 7, 2025
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 enhancement feature: object storage pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants