Skip to content

Conversation

@mladen-rusev-cyberark
Copy link
Collaborator

@mladen-rusev-cyberark mladen-rusev-cyberark commented Oct 23, 2025

The error message panic: unable to redefine 'v' shorthand in "version" flagset: it's already used for "verbose" flag clearly indicates that the shorthand -v was being defined for two different flags.

logs.AddFlags() function also defined a flag with the shorthand -v caused the conflict.

Using .BoolVar() instead of .BoolVarP() to define the verbose flag solves the problem.

Testing:

$ docker load < /home/mladen/IdeaProjects/jetstack-secure/_bin/scratch/image/oci-layout-ark.docker.tar
...
Loaded image: jetstack.local/disco-agent:v1.7.0-alpha.5-5-g37ae1dbc56bb50-dirty
$ docker images jetstack.local/disco-agent
REPOSITORY                   TAG                                      IMAGE ID       CREATED          SIZE
jetstack.local/disco-agent   v1.7.0-alpha.5-5-g37ae1dbc56bb50-dirty   1e87c0ca47e6   13 minutes ago   42.3MB
$ docker run --rm jetstack.local/disco-agent:v1.7.0-alpha.5-5-g37ae1dbc56bb50-dirty version
Preflight version:  v1.7.0-alpha.5-5-g37ae1dbc56bb50-dirty linux/amd64
$ docker run --rm jetstack.local/disco-agent:v1.7.0-alpha.5-5-g37ae1dbc56bb50-dirty version --verbose
Preflight version:  v1.7.0-alpha.5-5-g37ae1dbc56bb50-dirty linux/amd64
  Commit:  37ae1dbc56bb50ec667eeea8c6b2b8787c43e070
  Built:   2025-10-23-13:54:05-EEST
  Go:      go1.24.5

@inteon inteon merged commit 1d9e58c into master Oct 23, 2025
3 checks passed
@wallrj-cyberark wallrj-cyberark deleted the VC-45998 branch October 24, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants