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

Add apparmor #10017

Merged
merged 1 commit into from
Jan 10, 2015
Merged

Add apparmor #10017

merged 1 commit into from
Jan 10, 2015

Conversation

jessfraz
Copy link
Contributor

Docker-DCO-1.1-Signed-off-by: Jessica Frazelle jess@docker.com (github: jfrazelle)

@jessfraz
Copy link
Contributor Author

ping @tianon this IS drone fix

@tianon
Copy link
Member

tianon commented Jan 10, 2015

Ugh. AppArmor again. FML.

LGTM, begrudgingly

Docker-DCO-1.1-Signed-off-by: Jessica Frazelle <jess@docker.com> (github: jfrazelle)
@jessfraz
Copy link
Contributor Author

Seriously so hard to track down but the reason the daemon wasn't starting
was no file /sbin/apparmor_parser ughhhhh

On Friday, January 9, 2015, Tianon Gravi notifications@github.com wrote:

Ugh. AppArmor again. FML.

LGTM, begrudgingly


Reply to this email directly or view it on GitHub
#10017 (comment).

@tianon
Copy link
Member

tianon commented Jan 10, 2015

IT'S GREEN!

tianon added a commit that referenced this pull request Jan 10, 2015
Add apparmor package to Dockerfile
@tianon tianon merged commit 91b5634 into moby:master Jan 10, 2015
@jessfraz
Copy link
Contributor Author

Lol we updated the image on drone before ;) thanks for the fast merge phew

On Friday, January 9, 2015, Tianon Gravi notifications@github.com wrote:

Merged #10017 #10017.


Reply to this email directly or view it on GitHub
#10017 (comment).

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.

None yet

2 participants