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.

Sign up for GitHub

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

Jump to bottom

Cron should run every 5 mins #16865

Merged
merged 2 commits into from
Aug 26, 2019
Merged

Cron should run every 5 mins #16865

merged 2 commits into from
Aug 26, 2019

Conversation

nickvergessen
Copy link
Member

@nickvergessen
Cron should run every 5 mins
9b89bac 
So also 10 mins is now an error, so admins check their set up crons

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 17 milestone Aug 26, 2019
@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 Aug 26, 2019
@blizzz
Copy link
Member

blizzz commented Aug 26, 2019 •
edited
Loading

adjust tests please

@skjnldsv skjnldsv added 2. developing Work in progress and removed 4. to release Ready to be released and/or waiting for tests to finish labels Aug 26, 2019
@nickvergessen
Cleanup the Settings class
db968cb 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/5-min-cron branch from 7de31a8 to db968cb Compare August 26, 2019 09:50
@nickvergessen nickvergessen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 26, 2019
@nickvergessen
Copy link
Member Author

Done

@blizzz blizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 26, 2019
@rullzer rullzer merged commit 505f100 into master Aug 26, 2019
@rullzer rullzer deleted the bugfix/noid/5-min-cron branch August 26, 2019 17:29
@rullzer rullzer mentioned this pull request Aug 28, 2019
Merged
@kesselb
Copy link
Contributor

kesselb commented Sep 1, 2019

cc @enoch85 @kyrofa

enoch85 added a commit to nextcloud/vm that referenced this pull request Sep 1, 2019
@enoch85
change cron to 5 minutes
cf133de 
nextcloud/server#16865
@enoch85
Copy link
Member

enoch85 commented Sep 1, 2019

Thanks @kesselb!

enoch85 added a commit to nextcloud/vm that referenced this pull request Sep 3, 2019
@enoch85
change cron to 5 minutes
db9d465 
nextcloud/server#16865
Signed-off-by: enoch85 <github@hanssonit.se>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skjnldsv skjnldsv skjnldsv approved these changes

@blizzz blizzz blizzz approved these changes

@georgehrke georgehrke georgehrke approved these changes

@rullzer rullzer Awaiting requested review from rullzer

Assignees
No one assigned
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: settings
Projects
None yet
Milestone
Nextcloud 17
Development

Successfully merging this pull request may close these issues.

Documentation: backgroud jobs 15 min vs. 5 min
7 participants
@nickvergessen @blizzz @kesselb @enoch85 @georgehrke @skjnldsv @rullzer