New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

update call to NewProcessCollector #48

Open

olivierbeytrison wants to merge 1 commit into bakins:master from olivierbeytrison:fix-NewProcessCollector

olivierbeytrison commented Aug 17, 2023

While trying to build the code, got the following error :

./exporter.go:140:68: too many arguments in call to prometheus.NewProcessCollector
        have (int, string)
        want (prometheus.ProcessCollectorOpts)

Fixed by using the default struct


          update call to NewProcessCollector

39f0529

wmfgerrit pushed a commit to wikimedia/operations-docker-images-production-images that referenced this pull request


          prometheus-php-fpm-exporter: Bullseye update and fix build script

eba3df2

The original repo hasn't been updated in 9 months and contains a bug
breaking the build (see
bakins/php-fpm-exporter#48)

Use a fixed fork (https://github.com/olivierbeytrison/php-fpm-exporter)

Upgrade image to bullseye, build image to golang1.15

Change-Id: Ic22672202b9a573a3d12323f220d0554168e7bb7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet