Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

rootless: set sticky bit if using XDG_RUNTIME_DIR #1760

Merged
merged 1 commit into from
Mar 20, 2018
Merged

rootless: set sticky bit if using XDG_RUNTIME_DIR #1760

merged 1 commit into from
Mar 20, 2018

Conversation

cyphar
Copy link
Member

@cyphar cyphar commented Mar 19, 2018

According to the XDG specification1, in order to avoid the possibility of
our container states being auto-pruned every 6 hours we need to set the
sticky bit. Rather than handling all of the users of --root, we just
create the directory and set the sticky bit during detection, as it's
not expensive.

Fixes #1694
Signed-off-by: Aleksa Sarai asarai@suse.de

According to the XDG specification[1], in order to avoid the possibility of
our container states being auto-pruned every 6 hours we need to set the
sticky bit. Rather than handling all of the users of --root, we just
create the directory and set the sticky bit during detection, as it's
not expensive.

[1]: https://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html

Signed-off-by: Aleksa Sarai <asarai@suse.de>
@crosbymichael
Copy link
Member

crosbymichael commented Mar 19, 2018

LGTM

Approved with PullApprove

1 similar comment
@mrunalp
Copy link
Contributor

mrunalp commented Mar 20, 2018

LGTM

Approved with PullApprove

@mrunalp mrunalp merged commit 4e3f25b into opencontainers:master Mar 20, 2018
@cyphar cyphar deleted the rootless-xdg-sticky-bit branch March 21, 2018 07:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants