Skip to content

[master] Tear down FS between cron jobs (#26223)#1712

Merged
MorrisJobke merged 1 commit into
masterfrom
us_tear_down_fs_between_cron
Oct 12, 2016
Merged

[master] Tear down FS between cron jobs (#26223)#1712
MorrisJobke merged 1 commit into
masterfrom
us_tear_down_fs_between_cron

Conversation

@rullzer

@rullzer rullzer commented Oct 12, 2016

Copy link
Copy Markdown
Member

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
Copy Markdown

@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
Copy Markdown
Member

👍

@MorrisJobke

Copy link
Copy Markdown
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
Copy Markdown
Member

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

@karlitschek

Copy link
Copy Markdown
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.

5 participants