Skip to content
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.

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

Add php-fpm exporter #605

Merged
merged 1 commit into from
Mar 4, 2022
Merged

Conversation

kubicgruenfeld
Copy link

Pull Request (PR) description

This PR adds the php-fpm exporter from https://github.com/hipages/php-fpm_exporter

This Pull Request (PR) fixes the following issues

n/a

@kubicgruenfeld
Copy link
Author

Puppet 6 Tests on Ubuntu 16.04 and 18.04 are failing because of voxpupuli/beaker-hostgenerator#244

@kubicgruenfeld
Copy link
Author

I don't know why Archlinux is failing. But it's not related to my changes.
CC @bastelfreak

Copy link
Contributor

@baurmatt baurmatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please check the comment I've added.

(I will keep the PR unmerged for now because @kubicgruenfeld and I are colleagues. Please add an additional approve if you think this can be merged :))

# Not a big fan of copypasting but prometheus::daemon takes for granted
# a specific path embedded in the prometheus *_exporter tarball, which
# php-fpm_exporter lacks currently as of version 2.0.4
# TODO: patch prometheus::daemon to support custom extract directories
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there an open issue for this? If not, please open one.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@baurmatt baurmatt merged commit 6637238 into voxpupuli:master Mar 4, 2022
@baurmatt baurmatt changed the title add php-fpm exporter Add php-fpm exporter Mar 4, 2022
@baurmatt baurmatt added the enhancement New feature or request label Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants