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

Server has no maintenance window start time configured #2173

Closed
d3xt3r01 opened this issue Mar 4, 2024 · 8 comments
Closed

Server has no maintenance window start time configured #2173

d3xt3r01 opened this issue Mar 4, 2024 · 8 comments

Comments

@d3xt3r01
Copy link

d3xt3r01 commented Mar 4, 2024

Default image install yells this:

Can we add it to the default config?

Server has no maintenance window start time configured. This means resource intensive daily background jobs will also be executed during your main usage time. We recommend to set it to a time of low usage, so users are less impacted by the load caused from these heavy tasks. For more details see the documentation ↗.

@bonsje
Copy link

bonsje commented Mar 4, 2024

I had this issue also when setting up the nextcloud:latest (28.0.2), i hacked it away to give it the command in the docker directly. If read the documentation correctly yesterday you must give it a number for the starttime of the maintenance window to start. I can image that this could be a little difficult bc everybody is in different timezones and it was recommended to do this window in the night if i recall correctly (depends also on your size of the instance too). Maybe we could get an Environment Variable to get the maintenance window when we spin up the container? or maybe i read over it, but to my knowledge an maintenance window Environment Variable does not exist atm.

@J0WI
Copy link
Contributor

J0WI commented Mar 4, 2024

There is no default set by Nextcloud and therefore no default is set in the image. This is very similar to #1465: you can easily configure a value yourself that fits your needs (e.g. NC_maintenance_window_start: 1).

@J0WI J0WI pinned this issue Mar 4, 2024
@d3xt3r01
Copy link
Author

d3xt3r01 commented Mar 5, 2024

So we have some NEXTCLOUD_ variables, some NC_ variables and some that don't have anything ( like TRUSTED_PROXIES) ?

Some can be seen in sudo -u www-data php occ config:list (mainly NEXTCLOUD_ and TRUSTED_PROXIES) because they end up in the config file... but not the NC_ ones.

There's also no page in Nextcloud for admins to see the fully loaded/parsed settings or phpinfo() stuff to easily validate all the changes are propagated as they should... but that's another story.

@d3xt3r01
Copy link
Author

d3xt3r01 commented Mar 5, 2024

Also, if there isn't a cron daemon running in the container, is there a reason for maintenance_window_start to exist?

@J0WI
Copy link
Contributor

J0WI commented Mar 5, 2024

There is no default set by Nextcloud

There is a default and it means disabled:

Defaults to 100 which disables the feature

https://docs.nextcloud.com/server/28/admin_manual/configuration_server/config_sample_php_parameters.html#maintenance-window-start

So we have some NEXTCLOUD_ variables, some NC_ variables and some that don't have anything ( like TRUSTED_PROXIES) ?

Exactly.

Also, if there isn't a cron daemon running in the container, is there a reason for maintenance_window_start to exist?

I don't know. You may ask this on https://help.nextcloud.com/ or propose a change on https://github.com/nextcloud/server

@mattvisa
Copy link

Same. I get this error code on a default install of NextCloud in TrueNas Scale. If there is an update that can fix this by setting a default as described above for it to run at 3am based on local timezone, that would be great. I don't want to shell into the system and update if you will be pushing an update to fix.

@J0WI J0WI added the wontfix label Apr 24, 2024
@joshtrichards
Copy link
Member

This is an upstream matter (server). It doesn't make sense for us to do anything fancy merely in the Docker container for this parameter.

@joshtrichards joshtrichards closed this as not planned Won't fix, can't repro, duplicate, stale May 6, 2024
@joshtrichards
Copy link
Member

@d3xt3r01

Some can be seen in sudo -u www-data php occ config:list (mainly NEXTCLOUD_ and TRUSTED_PROXIES) because they end up in the config file... but not the NC_ ones.

FYI, the NC_ handling has now been fixed upstream: nextcloud/server#46140

@joshtrichards joshtrichards unpinned this issue Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants