Skip to content

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Oct 12, 2016

Because some cron jobs do not always properly clean up their FS usage
and others might not clean up before setting up the FS, this could
cause potential side effects.

To make sure we exclude side effects, we tear down the FS between cron
jobs.

From: owncloud/core#26257

CC: @nickvergessen @MorrisJobke @icewind1991

Because some cron jobs do not always properly clean up their FS usage
and others might not clean up before setting up the FS, this could
cause potential side effects.

To make sure we exclude side effects, we tear down the FS between cron
jobs.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added 3. to review Waiting for reviews downstream labels Oct 12, 2016
@rullzer rullzer added this to the Nextcloud 11.0 milestone Oct 12, 2016
@mention-bot
Copy link

@rullzer, thanks for your PR! By analyzing the history of the files in this pull request, we identified @butonic, @nickvergessen and @DeepDiver1975 to be potential reviewers.

@nickvergessen
Copy link
Member

👍

@MorrisJobke
Copy link
Member

Tested and works 👍

@MorrisJobke MorrisJobke merged commit 657f1af into master Oct 12, 2016
@MorrisJobke MorrisJobke deleted the us_tear_down_fs_between_cron branch October 12, 2016 08:00
@MorrisJobke
Copy link
Member

@karlitschek We should backport this to stable9 and stable10 to make cron jobs cause less side effects.

@karlitschek
Copy link
Member

please backport

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants