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.

Sign up for GitHub

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

Jump to bottom

Remove subjectaccessreviews.v1. hacks from test/cmd/authentication.sh #13158

Closed
enj opened this issue Feb 28, 2017 · 0 comments
Closed

Remove subjectaccessreviews.v1. hacks from test/cmd/authentication.sh #13158

enj opened this issue Feb 28, 2017 · 0 comments
Assignees
@enj
Labels
area/security area/techdebt priority/P2

Comments

@enj
Copy link
Contributor

enj commented Feb 28, 2017 •
edited
Loading

The discovery rest mapper does not correctly prioritize the core /v1 kube group. This will be fixed upstream in kubernetes/kubernetes#42275 and #13152 in origin. Once that is completed, we can revert the changes to test/cmd/authentication.sh: subjectaccessreviews.v1. -> subjectaccessreviews

xref: #13128
@enj enj mentioned this issue Feb 28, 2017
Merged
4 tasks
@abstractj abstractj mentioned this issue Jul 18, 2017
Merged
openshift-merge-robot added a commit that referenced this issue Jul 26, 2017
@openshift-merge-robot
Merge pull request #15314 from abstractj/issue-13158
9b71297 
Automatic merge from submit-queue (batch tested with PRs 15434, 15382, 15018, 15314, 15400)

Remove `subjectaccessreviews.v1.` hacks from `test/cmd/authentication.sh`

Fixes: #13158

[test]

//cc @enj
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@enj enj

Labels
area/security area/techdebt priority/P2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mfojtik @enj