version: add version string to metrics

[kkourt]

Before this patch:
tetragon_build_info{commit="61b575c72b22aa8bb6d7ed8af2d3dbfee43f6307",go_version="go1.23.1",modified="true",time="2024-10-23T06:20:18Z"} 1

After:
tetragon_build_info{commit="61b575c72b22aa8bb6d7ed8af2d3dbfee43f6307",go_version="go1.23.1",modified="true",time="2024-10-23T06:20:18Z",version="v1.3.0-pre.0-216-g61b575c72"} 1

metrics: add version to build information

[netlify]

✅ Deploy Preview for tetragon ready!

Name	Link
🔨 Latest commit	5394da8
🔍 Latest deploy log	https://app.netlify.com/sites/tetragon/deploys/6719150a336bd40008dd0f9f
😎 Deploy Preview	https://deploy-preview-3035--tetragon.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[mtardy]

Sure!! You just miss some metrics doc generation.

[kkourt]

Sure!! You just miss some metrics doc generation

I think the problem is that the docs are based on the SHA of the version, so every time I update it the SHA changes. I'll try to find a workaround.

[mtardy]

Sure!! You just miss some metrics doc generation

I think the problem is that the docs are based on the SHA of the version, so every time I update it the SHA changes. I'll try to find a workaround.

Oh yes put a constant in there :)

tetragon/cmd/tetragon-metrics-docs/metricsmd/metricsmd.go

Lines 31 to 49 in c0e513f

	Overrides: []metricsmd.LabelValues{
	{
	Label: "commit",
	Values: []string{"931b70f2c9878ba985ba6b589827bea17da6ec33"},
	},
	{
	Label: "go_version",
	Values: []string{"go1.22.0"},
	},
	{
	Label: "modified",
	Values: []string{"false"},
	},
	{
	Label: "time",
	Values: []string{"2022-05-13T15:54:45Z"},
	},
	},
	},

[kkourt]

Did something else. Happy to change it to what you suggested if it's an issue, but I think overriding Version also works.

[mtardy]

Did something else. Happy to change it to what you suggested if it's an issue, but I think overriding Version also works.

I think I would do it like Anna did with the rest of the values in there so that you don't touch the Makefile and we don't have two ways of doing the same thing.

[kkourt]

Did something else. Happy to change it to what you suggested if it's an issue, but I think overriding Version also works.

I think I would do it like Anna did with the rest of the values in there so that you don't touch the Makefile and we don't have two ways of doing the same thing.

Good point! Pushed a new version.

[mtardy]

Looks perfect to me like that

[lambdanis]

that's a useful label, thanks!