Incorrect version values from version.Get() about client-go HOT 6 CLOSED

How did you get that version? I assume with the version pkg? Well, it works as designed: the version package was designed to report the KubeVirt version, and values are set during the build process via ldflags. The values you see are the default values.
So even when you update your build process to set those values, it would report the version of your own component, and not client-go's version. Not sure if we can/want to support the latter. @rmohr wdyt?

You can get KubeVirt's server version if that helps:

virtClient, _ := kubecli.GetKubevirtClient()
info, _ := virtClient.ServerVersion().Get()

from client-go.

Not sure if we can/want to support the latter.

Yep I don't think it makes much sense to support this.

@slintes can we exclude anything related to version from client-go, when we sync with staging?

from client-go.

can we exclude anything related to version from client-go, when we sync with staging?

I think just excluding something won't work (well, it wouldn't be nice at least imho). But we could refactor it to only keep the version.Info struct in client-go (it's used in kubecli for getting the server's version), and move everything else back to kubevirt.

from client-go.

@slintes yep, you basically got it. Looks like I misunderstood the purpose of the version package in the client-go module.
I'd like a way to report (= print in the logs) the release of client-go which my binary has compiled in; I can think of few workarounds which don't require any change, so we can close this issue.

from client-go.

if you have a workaround which we could easily add to client-go, let us know 🙂

/close

from client-go.

@slintes: Closing this issue.

from client-go.