Add e2e tests for Authz v2 (#12271)#12814
Conversation
* WIP Authz v2 e2e tests * Add e2e tests for authz v2
|
@pitlv2109: The following tests failed, say
DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
diemtvu
left a comment
There was a problem hiding this comment.
Overall is ok. I just don't know should we enforce ourselves to have test in both e2e and integration folder or not.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: diemtvu, pitlv2109, yangminzhu If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/cc @andraxylia @costinm @GregHanson @kyessenov @ijsnellf @nmittler @rshriram @vadimeisenbergibm @ZackButcher for owner approval. This is the e2e test file for RBAC v2. |
|
I know that this is a cherry-pick, but we should probably be using the new test framework for new tests. |
|
Close this one and will create a new PR using the new testing framework + more tests. |
|
@pitlv2109: Closed this PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Cherry-pick from #12271
Add e2e tests for additional fields in ServiceRole and ServiceRoleBinding.
Add e2e tests for workload selector.
Add e2e tests for new CRD AuthorizationPolicy.
For #12394