Thread: Fix FTBFS for prometheus-sql-exporter

Hello All,

I've pushed some changes to prometheus-sql-exporter to address bug
#1067292 [1]. The Salsa CI pipelines are green. I'd like to request a
review and upload as cycles permit.

Let me know if anything needs changes.

Thanks,
--Bradford

[1]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1067292

Re: Bradford Boyle
> I've pushed some changes to prometheus-sql-exporter to address bug
> #1067292 [1]. The Salsa CI pipelines are green. I'd like to request a
> review and upload as cycles permit.

Hi Bradford,

thanks for the update, uploaded!

I had to revert the patch on the older distributions since go was
unhappy:

https://pgdgbuild.dus.dg-i.net/job/prometheus-sql-exporter-binaries/31/

I'm a bit worried that Ubuntu noble also needed the revert; we might
need to un-revert it there if they change the toolchain before the
release.


https://git.postgresql.org/gitweb/?p=pgapt.git;a=blobdiff;f=jenkins/generate-pgdg-source;h=1164b53400014b3275c7ac43bd7c39d1754a712e;hp=38db1e32a06e43a657aeab8c01f5f736efa1ae07;hb=753dc50b6b055494e0ef84d13a0dd8cdc0619680;hpb=2e6e6ce09c1d2c329b085618ef9be8665339b505

Christoph

Hi Christoph,

> I had to revert the patch on the older distributions since go was
> unhappy:

I forgot to mention it in my original email, but I thought the patch
might be an issue on older distributions. In the future, is there a way
I can trigger a snapshot build on pgdgbuild before notifying the mailing
list?

> I'm a bit worried that Ubuntu noble also needed the revert; we might
> need to un-revert it there if they change the toolchain before the
> release.

It looks like the replacement API is only available in
github.com/prometheus/client_golang@v1.19.0 and Ubuntu noble (as of
20240404) still is on 1.18.0. Unless 1.19.0 gets uploaded to noble, the
patch shouldn't need to be un-reverted. I will keep an eye on this.

--Bradford