Incorrect version reported on the `/metrics` endpoint about parca HOT 8 OPEN

It is coming from client_golang:

func NewBuildInfoCollector() Collector {
	path, version, sum := "unknown", "unknown", "unknown"
	if bi, ok := debug.ReadBuildInfo(); ok {
		path = bi.Main.Path
		version = bi.Main.Version
		sum = bi.Main.Sum
	}
	c := &selfCollector{MustNewConstMetric(
		NewDesc(
			"go_build_info",
			"Build information about the main Go module.",
			nil, Labels{"path": path, "version": version, "checksum": sum},
		),
		GaugeValue, 1)}
	c.init(c.self)
	return c
}

from parca.

Ok you beat me to it.

from parca.

I don't have a workaround at the moment. My window for fun ran out today ;-)

from parca.

I'll try to take a look at it over the weekend in my fun time :D

from parca.

Reading a bit more about how this is populated from: Hmm, I'm not sure I know how to get around this: https://pkg.go.dev/github.com/rb-go/promcolls/buildinfo

I'm not sure there is a trivial way to fix this given how Parca is built.

from parca.

Reading a bit more about how this is populated from: Hmm, I'm not sure I know how to get around this: https://pkg.go.dev/github.com/rb-go/promcolls/buildinfo

I'm not sure there is a trivial way to fix this given how Parca is built.

This is not a package we actually use 🤔

from parca.

Ah, okay, I think I misunderstood intitally. Must've got my wires crossed in Googling...!

Looking back at this line it looks like it actually goes back to here.

Looks like there is an ongoing issue here: golang/go#29228

from parca.

@jnsgruk If you have a workaround already, you can send a PR to client_golang until it is fixed in Go. I can review it (I'm a maintainer) and ensure it's shipped before the next release of client_golang (which should be in a couple of days).

from parca.