Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

selinux: Update to latest #3818

Merged
merged 2 commits into from
Jan 9, 2018
Merged

selinux: Update to latest #3818

merged 2 commits into from
Jan 9, 2018

Conversation

glevand
Copy link
Contributor

@glevand glevand commented Oct 4, 2017

Update the selinux package to the latest available. Moves selinux from pkg/selinux to vendor/github.com/opencontainers/selinux.

Fixes 'make unit-check' errors like the below when a directory named '/sys/fs/selinux' exists, but selinux is not enabled, such as when running in a Docker container without Docker selinux enabled.

--- FAIL: TestSetfilecon (0.00s)
selinux_test.go: Setfilecon failed
selinux_test.go: operation not supported

@glevand
Copy link
Contributor Author

glevand commented Oct 18, 2017

Rebased to latest.

Run glide-update.sh to refresh the vendor directory.

Signed-off-by: Geoff Levand <geoff@infradead.org>
Update the selinux package to the latest available.  Moves selinux
from pkg/selinux to vendor/github.com/opencontainers/selinux.

Fixes 'make unit-check' errors like these when a directory named
'/sys/fs/selinux' exists, but selinux is not enabled:

  --- FAIL: TestSetfilecon (0.00s)
        selinux_test.go: Setfilecon failed
        selinux_test.go: operation not supported

Signed-off-by: Geoff Levand <geoff@infradead.org>
@glevand
Copy link
Contributor Author

glevand commented Oct 26, 2017

Rebased to latest.

Copy link
Member

@iaguis iaguis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@iaguis iaguis merged commit 18fe845 into rkt:master Jan 9, 2018
@glevand glevand deleted the for-merge-selinux branch January 9, 2018 17:47
@iaguis iaguis added this to the 1.30.0 milestone Apr 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants