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

Huge load on maintenance task related to GP, following upgrade NC19 -> NC20, forcing to disable GP #1361

Closed
pierfu opened this issue Mar 8, 2021 · 13 comments
Labels
1. to develop Issues that are ready for development bug high high priority Nc20

Comments

@pierfu
Copy link

pierfu commented Mar 8, 2021

Hi,
a few days after upgrading Nextcloud from 19 to 20.0.7, we had to face a critical issue on a production server, ands it seamed directly related to GroupFolders.

  • huge lines in NC log, like ~5.000.000 characters, of failed requests, with :
    1/ recurring pattern : An exception occurred while executing 'DELETE FROM "oc_filecache_extended" WHERE "fileid" IN (?...?) ' with params [id1,... idN] :\n\nSQLSTATE[HY000]: General error: 7 number of parameters must be between 0 and 65535", ,"class":"Doctrine\DBAL\Driver\PDOException"},"DELETE FROM "oc_filecache_extended" WHERE "fileid" IN (?...?) ", [19017890,19537170,19459948,18911007,15822686,"And 89981 more entries
    2/ GroupFolders appears many times in these huge log lines : {"file":"/var/www/nextcloud/nextcloud-20.0.7/apps/groupfolders/lib/ACL/ACLManager.php","line":75, "function":"getRootStorageId","class":"OCA\GroupFolders\ACL\ACLManager","type":"->","args":[]}

  • problem was solved first times wirth "php occ maintenance:repair", that seamed to kill the job responsible for blocking everything, but never in a long-term way

  • the only long-term workaroud was to disable groupfolders completely

So,

  • has anyone known similar issue with after upgrade to Nextcloud 20 ?
  • could this be a maintenance issue with GroupFolders, which would require too much work to Nextcloud Core at the same time, instead of asking this maintenance with a maximum entries set ?

I join an excerpt of the log.
tail_-n1_nextcloud_abbr.log

@pierfu pierfu changed the title Huge load on mlaintenance task related to GP, following upgrade NC19 -> NC20, forcing to disable GP Huge load on maintenance task related to GP, following upgrade NC19 -> NC20, forcing to disable GP Mar 8, 2021
@pierreozoux pierreozoux added bug Nc20 blocking high high priority 0. Needs triage Issues that need to be triaged and removed blocking labels Mar 10, 2021
@brtptrs
Copy link

brtptrs commented Mar 18, 2021

We are seeing similar huge query statements, and many long running mysql requests that often include group-folders.
NC 20.0.7
I can not tell if the problem started with NC20. But we noticed problems recently, that wight coincide with the upgrade from 19 to 20.

@pierfu
Copy link
Author

pierfu commented Mar 26, 2021

Hi,
any update on this big issue ?

@ASerbinski
Copy link

That duplicate is not a duplicate. It involves php unnecessarily/inefficiently processing a long list of files in the trashcan.

@pierfu
Copy link
Author

pierfu commented Apr 8, 2021

Just to be clear, when i pubished this issue, by saying maintenance issue with GroupFolders i did not mean maintenance mode at all, but maintenance task run by cron or whatever.

@fschrempf fschrempf added 1. to develop Issues that are ready for development and removed 0. Needs triage Issues that need to be triaged labels Apr 30, 2021
@pierreozoux
Copy link
Member

@pierfu do you still have issues? Maybe an update fixed it?

@pierfu
Copy link
Author

pierfu commented Jun 15, 2021

@pierfu do you still have issues? Maybe an update fixed it?

I got no answer on this issue, and it happened on a big production instance, so the only thing i'm sure about is that i won't take that risk to just try and mybe break everything. Sorry :/

Is this extension planned on beeing merged in Nextcloud core ? I can see on https://apps.nextcloud.com/apps/groupfolders that Nextcloud GMBH is now editor. That could maybe help on maintaining this app more efficiently.

@pierreozoux
Copy link
Member

What do you mean nextcloud core?
It is part of the supported app:
https://docs.nextcloud.com/server/latest/admin_manual/installation/apps_supported.html

So if you have support with Nextcloud GMBH, they will gladly help you.

@pierfu
Copy link
Author

pierfu commented Jun 16, 2021

Thanks for your answer.
By core I mean not-an-app.

I missed this supported-apps page, and this possibility of asking support to Nextcloud GMBH.
But from what i undersand in pricing presentation, the only way of having quick support garanty (quick or not, what does mean : "Reaction time: 1h, 4h, 12h or 24h" ??), is Premium plan, but i definitly can't afford "100 users for €95.50/user/year" cost.
And your suggestion means that you can't rely on this Github issues, not really.

So i can't just try again GP and see if it crashes.
The really big problem with this app is that it changes data management too deeply : one cannot just try it, make GP shares, and disable it, because shares are not managed as they are in the core way.
I spent hours with my client on re-creating shares that were managed by GP, especially link shares.

@pierreozoux
Copy link
Member

I understand.
You could "duplicate" this instance in an other env to try.
And yes, the "community" support is for Home users mainly, it is not recommended to run big production instance without support or knowing what you are doing.
(But i'm not representing Nc here, I'm just part of the community, not an employee, and what I'm saying comes from my experience of Nextcloud communication)

@pierfu
Copy link
Author

pierfu commented Jun 17, 2021

Thanks Pierre,
we already have a staging instance, mainly for validating NC updates, but we never experienced nor could reproduce the issue on staging, surely because of scale and load difference.
So for now, i honestly can't recommend to re-enable GP, mainly because :

  • i had no technical answer on the issue
  • GP changes NC too deeply to be easy to enable/disable.

I think we can close even if not resolved.

@pierreozoux
Copy link
Member

There is a somehow related issue here:
#1215

As you want feel free to close or not, your call :)
(I think it is good to keep this issue open, it is documenting the current state of this app, but this is only my opinion :) )

@pierfu
Copy link
Author

pierfu commented Jun 18, 2021

OK, il leave it opened.

@solracsf
Copy link
Member

solracsf commented May 9, 2023

This was mainly fixed by 57cf151
Queries are now made in chunks to avoid that General error: 7 number of parameters must be between 0 and 65535

@solracsf solracsf closed this as completed May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Issues that are ready for development bug high high priority Nc20
Projects
None yet
Development

No branches or pull requests

6 participants