Merge pull request #18093 from soltysh/deprecate_secrets

Automatic merge from submit-queue (batch tested with PRs 17976, 17195, 18093, 18080, 17922).

Deprecate oc secrets subcomands

While recently working on the secrets issue I've noticed we have a set of `oc secrets` subcommands that can be easily replaced with `oc create secret` command. Thus, after discussing this topic with @juanvallejo and @deads2k I'm deprecating them.

/assign @juanvallejo @deads2k

Author: openshift-merge-robot

contrib/completions/bash/oc

@@ -16918,331 +16918,6 @@ _oc_secrets_link()
     noun_aliases=()
 }
 
-_oc_secrets_new()
-{
-    last_command="oc_secrets_new"
-    commands=()
-
-    flags=()
-    two_word_flags=()
-    local_nonpersistent_flags=()
-    flags_with_completion=()
-    flags_completion=()
-
-    flags+=("--allow-missing-template-keys")
-    local_nonpersistent_flags+=("--allow-missing-template-keys")
-    flags+=("--confirm")
-    local_nonpersistent_flags+=("--confirm")
-    flags+=("--no-headers")
-    local_nonpersistent_flags+=("--no-headers")
-    flags+=("--output=")
-    two_word_flags+=("-o")
-    local_nonpersistent_flags+=("--output=")
-    flags+=("--output-version=")
-    local_nonpersistent_flags+=("--output-version=")
-    flags+=("--quiet")
-    flags+=("-q")
-    local_nonpersistent_flags+=("--quiet")
-    flags+=("--show-all")
-    flags+=("-a")
-    local_nonpersistent_flags+=("--show-all")
-    flags+=("--show-labels")
-    local_nonpersistent_flags+=("--show-labels")
-    flags+=("--sort-by=")
-    local_nonpersistent_flags+=("--sort-by=")
-    flags+=("--template=")
-    flags_with_completion+=("--template")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--template=")
-    flags+=("--type=")
-    local_nonpersistent_flags+=("--type=")
-    flags+=("--as=")
-    flags+=("--as-group=")
-    flags+=("--cache-dir=")
-    flags+=("--certificate-authority=")
-    flags_with_completion+=("--certificate-authority")
-    flags_completion+=("_filedir")
-    flags+=("--client-certificate=")
-    flags_with_completion+=("--client-certificate")
-    flags_completion+=("_filedir")
-    flags+=("--client-key=")
-    flags_with_completion+=("--client-key")
-    flags_completion+=("_filedir")
-    flags+=("--cluster=")
-    flags+=("--config=")
-    flags_with_completion+=("--config")
-    flags_completion+=("_filedir")
-    flags+=("--context=")
-    flags+=("--insecure-skip-tls-verify")
-    flags+=("--log-flush-frequency=")
-    flags+=("--loglevel=")
-    flags+=("--logspec=")
-    flags+=("--match-server-version")
-    flags+=("--namespace=")
-    flags_with_completion+=("--namespace")
-    flags_completion+=("__oc_get_namespaces")
-    two_word_flags+=("-n")
-    flags_with_completion+=("-n")
-    flags_completion+=("__oc_get_namespaces")
-    flags+=("--request-timeout=")
-    flags+=("--server=")
-    flags+=("--token=")
-    flags+=("--user=")
-    flags+=("--v=")
-    flags+=("--vmodule=")
-
-    must_have_one_flag=()
-    must_have_one_noun=()
-    noun_aliases=()
-}
-
-_oc_secrets_new-basicauth()
-{
-    last_command="oc_secrets_new-basicauth"
-    commands=()
-
-    flags=()
-    two_word_flags=()
-    local_nonpersistent_flags=()
-    flags_with_completion=()
-    flags_completion=()
-
-    flags+=("--allow-missing-template-keys")
-    local_nonpersistent_flags+=("--allow-missing-template-keys")
-    flags+=("--ca-cert=")
-    flags_with_completion+=("--ca-cert")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--ca-cert=")
-    flags+=("--gitconfig=")
-    flags_with_completion+=("--gitconfig")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--gitconfig=")
-    flags+=("--no-headers")
-    local_nonpersistent_flags+=("--no-headers")
-    flags+=("--output=")
-    two_word_flags+=("-o")
-    local_nonpersistent_flags+=("--output=")
-    flags+=("--output-version=")
-    local_nonpersistent_flags+=("--output-version=")
-    flags+=("--password=")
-    local_nonpersistent_flags+=("--password=")
-    flags+=("--prompt")
-    local_nonpersistent_flags+=("--prompt")
-    flags+=("--show-all")
-    flags+=("-a")
-    local_nonpersistent_flags+=("--show-all")
-    flags+=("--show-labels")
-    local_nonpersistent_flags+=("--show-labels")
-    flags+=("--sort-by=")
-    local_nonpersistent_flags+=("--sort-by=")
-    flags+=("--template=")
-    flags_with_completion+=("--template")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--template=")
-    flags+=("--username=")
-    local_nonpersistent_flags+=("--username=")
-    flags+=("--as=")
-    flags+=("--as-group=")
-    flags+=("--cache-dir=")
-    flags+=("--certificate-authority=")
-    flags_with_completion+=("--certificate-authority")
-    flags_completion+=("_filedir")
-    flags+=("--client-certificate=")
-    flags_with_completion+=("--client-certificate")
-    flags_completion+=("_filedir")
-    flags+=("--client-key=")
-    flags_with_completion+=("--client-key")
-    flags_completion+=("_filedir")
-    flags+=("--cluster=")
-    flags+=("--config=")
-    flags_with_completion+=("--config")
-    flags_completion+=("_filedir")
-    flags+=("--context=")
-    flags+=("--insecure-skip-tls-verify")
-    flags+=("--log-flush-frequency=")
-    flags+=("--loglevel=")
-    flags+=("--logspec=")
-    flags+=("--match-server-version")
-    flags+=("--namespace=")
-    flags_with_completion+=("--namespace")
-    flags_completion+=("__oc_get_namespaces")
-    two_word_flags+=("-n")
-    flags_with_completion+=("-n")
-    flags_completion+=("__oc_get_namespaces")
-    flags+=("--request-timeout=")
-    flags+=("--server=")
-    flags+=("--token=")
-    flags+=("--user=")
-    flags+=("--v=")
-    flags+=("--vmodule=")
-
-    must_have_one_flag=()
-    must_have_one_noun=()
-    noun_aliases=()
-}
-
-_oc_secrets_new-dockercfg()
-{
-    last_command="oc_secrets_new-dockercfg"
-    commands=()
-
-    flags=()
-    two_word_flags=()
-    local_nonpersistent_flags=()
-    flags_with_completion=()
-    flags_completion=()
-
-    flags+=("--allow-missing-template-keys")
-    local_nonpersistent_flags+=("--allow-missing-template-keys")
-    flags+=("--docker-email=")
-    local_nonpersistent_flags+=("--docker-email=")
-    flags+=("--docker-password=")
-    local_nonpersistent_flags+=("--docker-password=")
-    flags+=("--docker-server=")
-    local_nonpersistent_flags+=("--docker-server=")
-    flags+=("--docker-username=")
-    local_nonpersistent_flags+=("--docker-username=")
-    flags+=("--no-headers")
-    local_nonpersistent_flags+=("--no-headers")
-    flags+=("--output=")
-    two_word_flags+=("-o")
-    local_nonpersistent_flags+=("--output=")
-    flags+=("--output-version=")
-    local_nonpersistent_flags+=("--output-version=")
-    flags+=("--show-all")
-    flags+=("-a")
-    local_nonpersistent_flags+=("--show-all")
-    flags+=("--show-labels")
-    local_nonpersistent_flags+=("--show-labels")
-    flags+=("--sort-by=")
-    local_nonpersistent_flags+=("--sort-by=")
-    flags+=("--template=")
-    flags_with_completion+=("--template")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--template=")
-    flags+=("--as=")
-    flags+=("--as-group=")
-    flags+=("--cache-dir=")
-    flags+=("--certificate-authority=")
-    flags_with_completion+=("--certificate-authority")
-    flags_completion+=("_filedir")
-    flags+=("--client-certificate=")
-    flags_with_completion+=("--client-certificate")
-    flags_completion+=("_filedir")
-    flags+=("--client-key=")
-    flags_with_completion+=("--client-key")
-    flags_completion+=("_filedir")
-    flags+=("--cluster=")
-    flags+=("--config=")
-    flags_with_completion+=("--config")
-    flags_completion+=("_filedir")
-    flags+=("--context=")
-    flags+=("--insecure-skip-tls-verify")
-    flags+=("--log-flush-frequency=")
-    flags+=("--loglevel=")
-    flags+=("--logspec=")
-    flags+=("--match-server-version")
-    flags+=("--namespace=")
-    flags_with_completion+=("--namespace")
-    flags_completion+=("__oc_get_namespaces")
-    two_word_flags+=("-n")
-    flags_with_completion+=("-n")
-    flags_completion+=("__oc_get_namespaces")
-    flags+=("--request-timeout=")
-    flags+=("--server=")
-    flags+=("--token=")
-    flags+=("--user=")
-    flags+=("--v=")
-    flags+=("--vmodule=")
-
-    must_have_one_flag=()
-    must_have_one_noun=()
-    noun_aliases=()
-}
-
-_oc_secrets_new-sshauth()
-{
-    last_command="oc_secrets_new-sshauth"
-    commands=()
-
-    flags=()
-    two_word_flags=()
-    local_nonpersistent_flags=()
-    flags_with_completion=()
-    flags_completion=()
-
-    flags+=("--allow-missing-template-keys")
-    local_nonpersistent_flags+=("--allow-missing-template-keys")
-    flags+=("--ca-cert=")
-    flags_with_completion+=("--ca-cert")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--ca-cert=")
-    flags+=("--gitconfig=")
-    flags_with_completion+=("--gitconfig")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--gitconfig=")
-    flags+=("--no-headers")
-    local_nonpersistent_flags+=("--no-headers")
-    flags+=("--output=")
-    two_word_flags+=("-o")
-    local_nonpersistent_flags+=("--output=")
-    flags+=("--output-version=")
-    local_nonpersistent_flags+=("--output-version=")
-    flags+=("--show-all")
-    flags+=("-a")
-    local_nonpersistent_flags+=("--show-all")
-    flags+=("--show-labels")
-    local_nonpersistent_flags+=("--show-labels")
-    flags+=("--sort-by=")
-    local_nonpersistent_flags+=("--sort-by=")
-    flags+=("--ssh-privatekey=")
-    flags_with_completion+=("--ssh-privatekey")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--ssh-privatekey=")
-    flags+=("--template=")
-    flags_with_completion+=("--template")
-    flags_completion+=("_filedir")
-    local_nonpersistent_flags+=("--template=")
-    flags+=("--as=")
-    flags+=("--as-group=")
-    flags+=("--cache-dir=")
-    flags+=("--certificate-authority=")
-    flags_with_completion+=("--certificate-authority")
-    flags_completion+=("_filedir")
-    flags+=("--client-certificate=")
-    flags_with_completion+=("--client-certificate")
-    flags_completion+=("_filedir")
-    flags+=("--client-key=")
-    flags_with_completion+=("--client-key")
-    flags_completion+=("_filedir")
-    flags+=("--cluster=")
-    flags+=("--config=")
-    flags_with_completion+=("--config")
-    flags_completion+=("_filedir")
-    flags+=("--context=")
-    flags+=("--insecure-skip-tls-verify")
-    flags+=("--log-flush-frequency=")
-    flags+=("--loglevel=")
-    flags+=("--logspec=")
-    flags+=("--match-server-version")
-    flags+=("--namespace=")
-    flags_with_completion+=("--namespace")
-    flags_completion+=("__oc_get_namespaces")
-    two_word_flags+=("-n")
-    flags_with_completion+=("-n")
-    flags_completion+=("__oc_get_namespaces")
-    flags+=("--request-timeout=")
-    flags+=("--server=")
-    flags+=("--token=")
-    flags+=("--user=")
-    flags+=("--v=")
-    flags+=("--vmodule=")
-
-    must_have_one_flag=()
-    must_have_one_noun=()
-    noun_aliases=()
-}
-
 _oc_secrets_unlink()
 {
     last_command="oc_secrets_unlink"
@@ -17300,10 +16975,6 @@ _oc_secrets()
     commands=()
     commands+=("add")
     commands+=("link")
-    commands+=("new")
-    commands+=("new-basicauth")
-    commands+=("new-dockercfg")
-    commands+=("new-sshauth")
     commands+=("unlink")
 
     flags=()