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.

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

[1.13.x] bump RunC / libcontainer to fix SELinux regression #30649

Merged
merged 1 commit into from
Feb 1, 2017

Conversation

thaJeztah
Copy link
Member

This bumps RunC and libcontainer to fix a regression
in 1.13 that caused SELinux labels to not be
applied correctly.

diff is http://github.com/docker/runc/compare/2f7393a47307a16f8cee44a37b262e8b81021e3e...9df8b306d01f59d3a8029be411de015b7304dd8f

For more information about this change, see the discussion on opencontainers/runc#1303, #30532 (comment), and opencontainers/runc#1109 (comment)

ping @cpuguy83 @runcom @rhatdan PTAL

@vieux
Copy link
Contributor

vieux commented Feb 1, 2017

ping @tonistiigi @crosbymichael

@@ -1,7 +1,9 @@
#!/bin/sh

TOMLV_COMMIT=9baf8a8a9f2ed20a8e54160840c492f937eeaf9a
RUNC_COMMIT=2f7393a47307a16f8cee44a37b262e8b81021e3e

# When updating RUNC_COMMIT, also update libcontainerd in vendor.conf accordingly
Copy link
Member

Choose a reason for hiding this comment

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

s/libcontainerd/runc/

Copy link
Member Author

Choose a reason for hiding this comment

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

ah, was doubting, because that line especially refers to libcontainer; will update, hold on

Copy link
Member Author

Choose a reason for hiding this comment

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

updated

This bumps RunC and libcontainer to fix a regression
in 1.13 that caused SELinux labels to not be
applied correctly.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@runcom
Copy link
Member

runcom commented Feb 1, 2017

LGTM

2 similar comments
@LK4D4
Copy link
Contributor

LK4D4 commented Feb 1, 2017

LGTM

@crosbymichael
Copy link
Contributor

LGTM

@vieux vieux merged commit 2556b7a into moby:1.13.x Feb 1, 2017
@thaJeztah thaJeztah deleted the 1.13-bump-runc branch February 1, 2017 23:14
@thaJeztah
Copy link
Member Author

I'll look at a similar PR for master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants