Skip to content

Commit

Permalink
fix local resource output oc set
Browse files Browse the repository at this point in the history
some `oc set ...` sub-commands do not implement `--dry-run` or `--local`
options. This patch adds these missing options to all sub-commands and
ensures that backwards compatibility remains for commands that treat the
--output option as a ``dry-run flag.

```
$ oc set image dc dctest dctest-1=default/testimage:latest --dry-run
NAME      REVISION   DESIRED   CURRENT   TRIGGERED BY
dctest    1          1         1         config

$ oc set image dc dctest dctest-1=default/testimage:latest -o name
deploymentconfig/dctest

$ oc set build-secret mysecret --push --all -o
custom-columns=NAME:.metadata.name
NAME
gitauthtest
repo-base

$ oc set build-secret mysecret --push --all --dry-run
NAME          TYPE      FROM      LATEST
gitauthtest   Source    Git       1
NAME        TYPE      FROM      LATEST
repo-base   Docker    Git       8
```
  • Loading branch information
juanvallejo committed May 10, 2017
1 parent 31caa0d commit 763e6ec
Show file tree
Hide file tree
Showing 15 changed files with 407 additions and 138 deletions.
36 changes: 36 additions & 0 deletions contrib/completions/bash/oc
Original file line number Diff line number Diff line change
Expand Up @@ -10532,6 +10532,8 @@ _oc_env()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--env=")
two_word_flags+=("-e")
local_nonpersistent_flags+=("--env=")
Expand All @@ -10546,6 +10548,8 @@ _oc_env()
local_nonpersistent_flags+=("--from=")
flags+=("--list")
local_nonpersistent_flags+=("--list")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -15271,13 +15275,17 @@ _oc_set_build-hook()
local_nonpersistent_flags+=("--allow-missing-template-keys")
flags+=("--command")
local_nonpersistent_flags+=("--command")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
two_word_flags+=("-f")
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -15354,6 +15362,8 @@ _oc_set_build-secret()
local_nonpersistent_flags+=("--all")
flags+=("--allow-missing-template-keys")
local_nonpersistent_flags+=("--allow-missing-template-keys")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
Expand Down Expand Up @@ -15444,6 +15454,8 @@ _oc_set_deployment-hook()
flags+=("--container=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--container=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--environment=")
two_word_flags+=("-e")
local_nonpersistent_flags+=("--environment=")
Expand Down Expand Up @@ -15542,6 +15554,8 @@ _oc_set_env()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--env=")
two_word_flags+=("-e")
local_nonpersistent_flags+=("--env=")
Expand All @@ -15556,6 +15570,8 @@ _oc_set_env()
local_nonpersistent_flags+=("--from=")
flags+=("--list")
local_nonpersistent_flags+=("--list")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -15725,6 +15741,8 @@ _oc_set_probe()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--failure-threshold=")
local_nonpersistent_flags+=("--failure-threshold=")
flags+=("--filename=")
Expand All @@ -15740,6 +15758,8 @@ _oc_set_probe()
local_nonpersistent_flags+=("--initial-delay-seconds=")
flags+=("--liveness")
local_nonpersistent_flags+=("--liveness")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--open-tcp=")
Expand Down Expand Up @@ -15917,6 +15937,8 @@ _oc_set_route-backends()
local_nonpersistent_flags+=("--all")
flags+=("--allow-missing-template-keys")
local_nonpersistent_flags+=("--allow-missing-template-keys")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--equal")
local_nonpersistent_flags+=("--equal")
flags+=("--filename=")
Expand All @@ -15926,6 +15948,8 @@ _oc_set_route-backends()
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -16003,6 +16027,8 @@ _oc_set_triggers()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
Expand All @@ -16024,6 +16050,8 @@ _oc_set_triggers()
local_nonpersistent_flags+=("--from-webhook")
flags+=("--from-webhook-allow-env")
local_nonpersistent_flags+=("--from-webhook-allow-env")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--manual")
local_nonpersistent_flags+=("--manual")
flags+=("--no-headers")
Expand Down Expand Up @@ -16119,13 +16147,17 @@ _oc_set_volumes()
local_nonpersistent_flags+=("--containers=")
flags+=("--default-mode=")
local_nonpersistent_flags+=("--default-mode=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
two_word_flags+=("-f")
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--mount-path=")
two_word_flags+=("-m")
local_nonpersistent_flags+=("--mount-path=")
Expand Down Expand Up @@ -16584,13 +16616,17 @@ _oc_volumes()
local_nonpersistent_flags+=("--containers=")
flags+=("--default-mode=")
local_nonpersistent_flags+=("--default-mode=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
two_word_flags+=("-f")
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--mount-path=")
two_word_flags+=("-m")
local_nonpersistent_flags+=("--mount-path=")
Expand Down
36 changes: 36 additions & 0 deletions contrib/completions/bash/openshift
Original file line number Diff line number Diff line change
Expand Up @@ -15804,6 +15804,8 @@ _openshift_cli_env()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--env=")
two_word_flags+=("-e")
local_nonpersistent_flags+=("--env=")
Expand All @@ -15818,6 +15820,8 @@ _openshift_cli_env()
local_nonpersistent_flags+=("--from=")
flags+=("--list")
local_nonpersistent_flags+=("--list")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -20671,13 +20675,17 @@ _openshift_cli_set_build-hook()
local_nonpersistent_flags+=("--allow-missing-template-keys")
flags+=("--command")
local_nonpersistent_flags+=("--command")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
two_word_flags+=("-f")
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -20756,6 +20764,8 @@ _openshift_cli_set_build-secret()
local_nonpersistent_flags+=("--all")
flags+=("--allow-missing-template-keys")
local_nonpersistent_flags+=("--allow-missing-template-keys")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
Expand Down Expand Up @@ -20848,6 +20858,8 @@ _openshift_cli_set_deployment-hook()
flags+=("--container=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--container=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--environment=")
two_word_flags+=("-e")
local_nonpersistent_flags+=("--environment=")
Expand Down Expand Up @@ -20948,6 +20960,8 @@ _openshift_cli_set_env()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--env=")
two_word_flags+=("-e")
local_nonpersistent_flags+=("--env=")
Expand All @@ -20962,6 +20976,8 @@ _openshift_cli_set_env()
local_nonpersistent_flags+=("--from=")
flags+=("--list")
local_nonpersistent_flags+=("--list")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -21135,6 +21151,8 @@ _openshift_cli_set_probe()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--failure-threshold=")
local_nonpersistent_flags+=("--failure-threshold=")
flags+=("--filename=")
Expand All @@ -21150,6 +21168,8 @@ _openshift_cli_set_probe()
local_nonpersistent_flags+=("--initial-delay-seconds=")
flags+=("--liveness")
local_nonpersistent_flags+=("--liveness")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--open-tcp=")
Expand Down Expand Up @@ -21331,6 +21351,8 @@ _openshift_cli_set_route-backends()
local_nonpersistent_flags+=("--all")
flags+=("--allow-missing-template-keys")
local_nonpersistent_flags+=("--allow-missing-template-keys")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--equal")
local_nonpersistent_flags+=("--equal")
flags+=("--filename=")
Expand All @@ -21340,6 +21362,8 @@ _openshift_cli_set_route-backends()
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--no-headers")
local_nonpersistent_flags+=("--no-headers")
flags+=("--output=")
Expand Down Expand Up @@ -21419,6 +21443,8 @@ _openshift_cli_set_triggers()
flags+=("--containers=")
two_word_flags+=("-c")
local_nonpersistent_flags+=("--containers=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
Expand All @@ -21440,6 +21466,8 @@ _openshift_cli_set_triggers()
local_nonpersistent_flags+=("--from-webhook")
flags+=("--from-webhook-allow-env")
local_nonpersistent_flags+=("--from-webhook-allow-env")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--manual")
local_nonpersistent_flags+=("--manual")
flags+=("--no-headers")
Expand Down Expand Up @@ -21537,13 +21565,17 @@ _openshift_cli_set_volumes()
local_nonpersistent_flags+=("--containers=")
flags+=("--default-mode=")
local_nonpersistent_flags+=("--default-mode=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
two_word_flags+=("-f")
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--mount-path=")
two_word_flags+=("-m")
local_nonpersistent_flags+=("--mount-path=")
Expand Down Expand Up @@ -21967,13 +21999,17 @@ _openshift_cli_volumes()
local_nonpersistent_flags+=("--containers=")
flags+=("--default-mode=")
local_nonpersistent_flags+=("--default-mode=")
flags+=("--dry-run")
local_nonpersistent_flags+=("--dry-run")
flags+=("--filename=")
flags_with_completion+=("--filename")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
two_word_flags+=("-f")
flags_with_completion+=("-f")
flags_completion+=("__handle_filename_extension_flag yaml|yml|json")
local_nonpersistent_flags+=("--filename=")
flags+=("--local")
local_nonpersistent_flags+=("--local")
flags+=("--mount-path=")
two_word_flags+=("-m")
local_nonpersistent_flags+=("--mount-path=")
Expand Down
Loading

0 comments on commit 763e6ec

Please sign in to comment.