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

oc adm policy subcommands don't have -o or --dry-run #14808

Closed
stevekuznetsov opened this issue Jun 21, 2017 · 0 comments
Closed

oc adm policy subcommands don't have -o or --dry-run #14808

stevekuznetsov opened this issue Jun 21, 2017 · 0 comments
Assignees
Labels
area/usability component/cli kind/bug Categorizes issue or PR as related to a bug. priority/P2

Comments

@stevekuznetsov
Copy link
Contributor

Same motivation as #14807 -- we should let people use the client to create their YAML.

/cc @deads2k @juanvallejo @fabianofranz

@stevekuznetsov stevekuznetsov added area/usability component/cli kind/bug Categorizes issue or PR as related to a bug. priority/P2 labels Jun 21, 2017
@juanvallejo juanvallejo self-assigned this Aug 4, 2017
openshift-merge-robot added a commit that referenced this issue Sep 23, 2017
…n-oc-adm-policy

Automatic merge from submit-queue (batch tested with PRs 16439, 16218, 16387, 16509)

add --output & --dry-run options to oc-adm-policy sub-commands  

Fixes #14808

Adds `--output` flags with support for json and yaml output to every `oc adm policy` sub-command, as well as `--dry-run` flag support to several sub-commands.

~~Still have a few sub-commands left to update, but opening PR to begin gathering any feedback.~~

cc @openshift/cli-review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/usability component/cli kind/bug Categorizes issue or PR as related to a bug. priority/P2
Projects
None yet
Development

No branches or pull requests

3 participants