operator-registry icon indicating copy to clipboard operation
operator-registry copied to clipboard

opm index -- refactor global flags support

Open grokspawn opened this issue 3 years ago • 0 comments

in dialog https://github.com/operator-framework/operator-registry/pull/986#discussion_r923869446 we learned that opm index sets help/usage support for 'global flags' as persistent cobra flags.

Since opm index is to be deprecated because it only handles a deprecated SQLite index format, we need to refactor the flags support.

Instead of using persistent flags set at the opm root, we need to have each subcommand locally implement support for the use-http and skip-tls-verify.

Preliminary file list (under cmd hierarchy):

  • ./opm/internal/util/util.go
  • ./opm/render/cmd.go
  • ./opm/registry/add.go
  • ./opm/index/cmd.go
  • ./opm/alpha/veneer/basic.go
  • ./opm/alpha/bundle/unpack.go

grokspawn avatar Jul 26 '22 18:07 grokspawn