Package: prometheus-ipmi-exporter
Severity: serious
Tags: patch ftbfs
Justification: fails to build from source (but built successfully in the past)
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu lunar ubuntu-patch

Dear Maintainer,

Recent API changes in the github.com/prometheus/exporter-toolkit/web
package, which are included in golang-github-prometheus-exporter-toolkit
0.8.2-1, cause the autopkgtest (and re-build) to fail.

This can be fixed by an upstream commit that adapts to the new API.

Thanks,
Nick
diff -Nru 
prometheus-ipmi-exporter-1.6.1/debian/patches/02-fix-build-with-new-exporter-toolkit.patch
 
prometheus-ipmi-exporter-1.6.1/debian/patches/02-fix-build-with-new-exporter-toolkit.patch
--- 
prometheus-ipmi-exporter-1.6.1/debian/patches/02-fix-build-with-new-exporter-toolkit.patch
  1969-12-31 19:00:00.000000000 -0500
+++ 
prometheus-ipmi-exporter-1.6.1/debian/patches/02-fix-build-with-new-exporter-toolkit.patch
  2022-12-13 15:51:36.000000000 -0500
@@ -0,0 +1,33 @@
+Description: Fix build with golang-github-prometheus-exporter-toolkit 0.8.2
+Origin: upstream, 
https://github.com/prometheus-community/ipmi_exporter/commit/e8989cfdf79bebb2e3d84e43f65a7fc027ee7bba.patch
+Bug-Ubuntu: https://launchpad.net/bugs/1999574
+Last-Update: 2022-12-13
+---
+--- a/main.go
++++ b/main.go
+@@ -41,11 +41,7 @@
+               "freeipmi.path",
+               "Path to FreeIPMI executables.",
+       ).Default("/usr/sbin").String()
+-      listenAddress = kingpin.Flag(
+-              "web.listen-address",
+-              "Address to listen on for web interface and telemetry.",
+-      ).Default(":9290").String()
+-      webConfig = webflag.AddFlags(kingpin.CommandLine)
++      webConfig = webflag.AddFlags(kingpin.CommandLine, ":9290")
+ 
+       sc = &SafeConfig{
+               C: &Config{},
+@@ -169,10 +165,8 @@
+             </html>`))
+       })
+ 
+-      level.Info(logger).Log("msg", "Listening on", "address", *listenAddress)
+-
+-      srv := &http.Server{Addr: *listenAddress}
+-      if err := web.ListenAndServe(srv, *webConfig, logger); err != nil {
++      srv := &http.Server{}
++      if err := web.ListenAndServe(srv, webConfig, logger); err != nil {
+               level.Error(logger).Log("msg", "HTTP listener stopped", 
"error", err)
+               os.Exit(1)
+       }
diff -Nru prometheus-ipmi-exporter-1.6.1/debian/patches/series 
prometheus-ipmi-exporter-1.6.1/debian/patches/series
--- prometheus-ipmi-exporter-1.6.1/debian/patches/series        2022-06-22 
07:19:56.000000000 -0400
+++ prometheus-ipmi-exporter-1.6.1/debian/patches/series        2022-12-13 
15:23:10.000000000 -0500
@@ -1 +1,2 @@
 01-debian-defaults.patch
+02-fix-build-with-new-exporter-toolkit.patch

Reply via email to