Skip to content

oc policy can-i --list output is not parser friendly. #11147

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
Closed
pruan-rht opened this issue Sep 28, 2016 · 4 comments · Fixed by #15125
Closed

oc policy can-i --list output is not parser friendly. #11147

pruan-rht opened this issue Sep 28, 2016 · 4 comments · Fixed by #15125
Assignees
@abstractj @juanvallejo
Labels
component/auth component/cli priority/P3

Comments

@pruan-rht
Copy link
Member

oc policy can-i --list output is not parser friendly due to the lack of --output=format support

Version

oc v1.3.0-rc1+ea6ca28

Steps To Reproduce

oc policy can-i --list

Current Result

not parser friendly

Expected Result

parser friendly.
@abstractj abstractj assigned abstractj and unassigned fabianofranz Jun 27, 2017
@fabianofranz
Copy link
Member

Hey @liggitt @deads2k, @abstractj is taking this one. Which output formats would you expect to have here? If the entries directly reflects single API objects then I'd say all output formats we have in get probably apply; otherwise maybe just json and yaml?

@deads2k
Copy link
Contributor

deads2k commented Jun 27, 2017

It doesn't have a name with the object comes back. Or at least not a meaningful one. Otherwise, it is a normal API object.

@abstractj
Copy link

@fabianofranz We've been discussing some changes here https://github.com/abstractj/origin/commit/88bf1621ec7b4ea8e31f1bedd4c4336c6969f3a1#commitcomment-22954041. Sharing just in case.

@abstractj abstractj assigned juanvallejo and unassigned abstractj Jul 10, 2017
@abstractj
Copy link

@fabianofranz fyi I reassigned this to Juan, because his solution (juanvallejo@875b22c) is way ahead than mine

@abstractj abstractj mentioned this issue Jul 10, 2017
Merged
@abstractj abstractj self-assigned this Jul 21, 2017
openshift-merge-robot added a commit that referenced this issue Aug 2, 2017
@openshift-merge-robot
Merge pull request #15125 from abstractj/issue-11147
68da586 
Automatic merge from submit-queue (batch tested with PRs 15512, 15582, 15125, 15590, 15593)

oc policy can-i --list output is not parser friendly

@fabianofranz this fixes #11147 is just a rebase from the changes that @juanvallejo made. 
He pretty much did all the hard work on it.

fixes #11147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abstractj abstractj

@juanvallejo juanvallejo

Labels
component/auth component/cli priority/P3
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

oc policy can-i --list output is not parser friendly
6 participants
@abstractj @fabianofranz @danmcp @pruan-rht @juanvallejo @deads2k