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

validate: add selinux smoke tests #200

Merged
merged 1 commit into from
Nov 22, 2017

Conversation

runcom
Copy link
Contributor

@runcom runcom commented Nov 21, 2017

Add basic selinux tests about options validation. Will follow up with real checks.

Part of #68
@feiskyer @Random-Liu @mrunalp PTAL

Signed-off-by: Antonio Murdaca runcom@redhat.com

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 21, 2017
@runcom runcom force-pushed the add-selinux-validation branch from 273ebbc to 32b006b Compare November 21, 2017 09:04
Signed-off-by: Antonio Murdaca <runcom@redhat.com>
@runcom runcom force-pushed the add-selinux-validation branch from 32b006b to 83aebdd Compare November 21, 2017 09:15
Copy link
Member

@mrunalp mrunalp left a comment

Choose a reason for hiding this comment

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

LGTM

@mrunalp
Copy link
Member

mrunalp commented Nov 21, 2017

@Random-Liu @feiskyer PTAL

Copy link
Member

@feiskyer feiskyer left a comment

Choose a reason for hiding this comment

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

@runcom Thanks for adding the tests. LGTM

@feiskyer feiskyer added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 22, 2017
@feiskyer feiskyer merged commit a9e38a4 into kubernetes-sigs:master Nov 22, 2017
@runcom runcom deleted the add-selinux-validation branch November 22, 2017 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants