Release new version to get newer docker image
Closed this issue · 6 comments
I see a new main image got built after v1.5.0 has been built quite a while ago (10 months).
Would it be possible to bump the version also to get a newer docker image? (Your Go Bump had some security fixes e.g. golang/go#67680 and I'd like to see them in an image that can be referenced in the official helm chart - https://github.com/prometheus-community/helm-charts/tree/main/charts/alertmanager-snmp-notifier )
Good point indeed. I just released the v1.6.0 that uses the latest Golang images.
$ docker run maxwo/snmp-notifier:v1.6.0 --version
snmp_notifier, version 1.6.0 (branch: HEAD, revision: a7a4593a9db0b46746403d0d338518111a0aabc8)
build user: root@e85efbf57b23
build date: 20240929-08:25:40
go version: go1.23.1
platform: linux/arm64
tags: netgoI will update the chart to reference that new version. Until then, you can customize the image.tag value of the current chart to take it into account.
Good point indeed. I just released the v1.6.0 that uses the latest Golang images.
$ docker run maxwo/snmp-notifier:v1.6.0 --version snmp_notifier, version 1.6.0 (branch: HEAD, revision: a7a4593a9db0b46746403d0d338518111a0aabc8) build user: root@e85efbf57b23 build date: 20240929-08:25:40 go version: go1.23.1 platform: linux/arm64 tags: netgoI will update the chart to reference that new version. Until then, you can customize the
image.tagvalue of the current chart to take it into account.
Thanks! FYI: The chart automatically uses the appversion so if the appversion is bumped, it should be sufficient!
Yes, but I was willing to add another feature in the chart. I will do a simple PR just for the appVersion though.
Helm chart PR: prometheus-community/helm-charts#4893
Needs to be merged, but that ends that issue. Thank you
FYI, Helm Chart PR has been merged.
FYI, Helm Chart PR has been merged.
Thanks! I was also monitoring this, great work!