markormesher / iperf-prometheus-collector

GNU Affero General Public License v3.0
3 stars 1 forks source link

full rewrite #35

Closed markormesher closed 1 day ago

markormesher commented 1 day ago

FYI @gberche-orange, I've decided to nuke this project and re-write it in Go - I'm working on a lot of automation for Go that will keep it more up to date in the future, and it's more suitable for a small utility like this. I'll incorporate a lot of the feedback from your issues in this version over the next few days before I tag a test release for you to try out.

markormesher commented 1 day ago

@gberche-orange Available now to test here: https://github.com/markormesher/iperf-prometheus-collector/pkgs/container/iperf-prometheus-collector/291343138?tag=v0.3.0-rc0. Config is as-per the README now merged to main.

gberche-orange commented 3 hours ago

@markormesher I'm observing a weird error when running the container from ghcr.io/markormesher/iperf-prometheus-collector:v0.3.0-rc0 : the cmd exits with 127 error code, and message sh: /app/build/main: not found

Here is the output of my interactive command within the container

$ k exec -it deployment.apps/iperf-exporter-from-r1-z2-private-to-r1-z2-private-protocol-tcp -- /bin/sh
/app #  /app/build/main
/bin/sh: /app/build/main: not found
/app # echo $?
127
/app # ls -al /app/build/main
-rwxr-xr-x    1 root     root       7684132 Oct 18 08:45 /app/build/main
/app # id
uid=0(root) gid=0(root) groups=0(root),1(bin),2(daemon),3(sys),4(adm),6(disk),10(wheel),11(floppy),20(dialout),26(tape),27(video)

I'll now look deeper in the container file, and run the build locally to try to further diagnose the issue.

markormesher commented 1 hour ago

That's odd - I'll take a look at the container image if I get some time on my laptop today and see what's going on.

gberche-orange commented 31 minutes ago

This post with similar symptom makes be suspect missing glibc/libs in the golang-alpine image. https://pet2cattle.com/2022/11/alpine-not-found

I'm trying out with FROM debian:11-slim as suggested in the post

markormesher commented 27 minutes ago

Ah that's likely. If that works I'll change the main branch over to a Deb based image.

gberche-orange commented 25 minutes ago

argn, Not better alpine:3.20 when running the command within the dockerfile, it still fails

https://github.com/orange-cloudfoundry/iperf-prometheus-collector/actions/runs/11417749415/job/31770391937#step:5:238

image

https://github.com/orange-cloudfoundry/iperf-prometheus-collector/blob/67547dc9ab478fe01424e1c9c1fc960008087c70/Dockerfile#L18

gberche-orange commented 23 minutes ago

Here is the modified docker file https://github.com/orange-cloudfoundry/iperf-prometheus-collector/blob/67547dc9ab478fe01424e1c9c1fc960008087c70/Dockerfile#L18