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

Enable APC in cron job #117

Merged
merged 1 commit into from Jul 2, 2021
Merged

Conversation

danb35
Copy link
Contributor

@danb35 danb35 commented Jun 24, 2021

Nextcloud docs recommend enabling APC for the periodic cron job to avoid out of memory errors. See https://community.nethserver.org/t/nextcloud-cron-issue/18495/2?u=danb35

Nextcloud docs recommend enabling APC for the periodic cron job to avoid out of memory errors.  See https://community.nethserver.org/t/nextcloud-cron-issue/18495/2?u=danb35
Copy link
Member

@gsanchietti gsanchietti left a comment

Choose a reason for hiding this comment

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

Thank you!

Do you think we should do the same for the occ wrapper https://github.com/NethServer/nethserver-nextcloud/blob/master/root/usr/local/sbin/occ?

@danb35
Copy link
Contributor Author

danb35 commented Jun 24, 2021

Hard to say. I tend to think it wouldn't hurt, but I don't recall seeing reports of a problem with the occ command the way that some (but not all) are seeing with the cron job. The counterpoint would be that the PHP docs kind of recommend against having APC enabled at the CLI (see https://www.php.net/manual/en/apcu.configuration.php), so I'd probably be inclined not to unless we were seeing out of memory problems with the occ command too.

@gsanchietti
Copy link
Member

We enabled APC inside the migration code, because of the memory error.
Still I hadn't the same problem when running occ from the command line.

In the end, I agree with you: let's keep occ wrapper untouched for now.

@gsanchietti
Copy link
Member

Ready for NethServer/dev#6539

@gsanchietti gsanchietti merged commit e26c97b into NethServer:master Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants