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

Test for new CRI features #131

Closed
2 tasks done
Random-Liu opened this issue Sep 6, 2017 · 8 comments
Closed
2 tasks done

Test for new CRI features #131

Random-Liu opened this issue Sep 6, 2017 · 8 comments
Assignees

Comments

@Random-Liu
Copy link
Contributor

Random-Liu commented Sep 6, 2017

We need CRI validation test for new CRI features:

We should at least add test for 2).

@Random-Liu
Copy link
Contributor Author

/cc @miaoyq

@miaoyq
Copy link
Contributor

miaoyq commented Sep 7, 2017

@Random-Liu OK, I will do my best to finish this. : )

@Random-Liu
Copy link
Contributor Author

Random-Liu commented Sep 7, 2017

@miaoyq I think only do 2) is good enough for now. :)

For 1) we do have a node e2e test https://github.com/kubernetes/kubernetes/blob/master/test/e2e_node/security_context_test.go#L366, we could migrate some test here.

@feiskyer
Copy link
Member

@Random-Liu @miaoyq updated descriptions to include a list of todos.

@feiskyer
Copy link
Member

@miaoyq Are you working on Mount propagation tests?

@miaoyq
Copy link
Contributor

miaoyq commented Sep 20, 2017

@feiskyer Yes, have finished the coding, testing now.

@feiskyer
Copy link
Member

@miaoyq cool. I will work on NoNewPrivilege.

@feiskyer feiskyer self-assigned this Sep 20, 2017
miaoyq pushed a commit to miaoyq/cri-tools that referenced this issue Sep 20, 2017
@feiskyer
Copy link
Member

all done.

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

No branches or pull requests

3 participants