Skip to content

Conversation

@mjura
Copy link
Collaborator

@mjura mjura commented Oct 11, 2021

Adding support for Docker containers and enforce lockc profiles.
Example usage can be:

docker run --label org.lockc.policy=privileged [image]

Fixes: #57

Copy link
Member

@vadorovsky vadorovsky left a comment

Choose a reason for hiding this comment

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

The code looks good. Good job! 💯

I would edit the commit message a bit. I don't think that the following info is useful:

(cherry picked from commit 2dcad30)

it's some commit from a closed PR. I would remove that line and add Fixes: #57 somewhere to automatically close the issue once the PR is merged.

@mjura mjura force-pushed the docker-support branch 2 times, most recently from bb30f54 to 40e60a7 Compare October 11, 2021 11:52
Adding support for Docker containers and enforce lockc profiles.
Example usage can be:

docker run --label org.lockc.policy=privileged [image]

Fixes: lockc-project#57

Signed-off-by: Michal Jura <[email protected]>
@vadorovsky vadorovsky merged commit 24ee17b into lockc-project:main Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docker: policy level choice

2 participants