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 varnish_exporter #171

Merged
merged 7 commits into from
Apr 5, 2018
Merged

add varnish_exporter #171

merged 7 commits into from
Apr 5, 2018

Conversation

blupman
Copy link

@blupman blupman commented Mar 14, 2018

I have used haproxy_exporter and rabbitmq_exporter to create varnish_exporter,

it installs https://github.com/jonnenauha/prometheus_varnish_exporter/ which seems to be a commonly used varnish_exporter

@bastelfreak
Copy link
Member

Hi @blupman, thanks for your PR. Can you please check the used email address in your commit? It isn't associated with your github account.

@bastelfreak bastelfreak added enhancement New feature or request needs-rebase labels Mar 17, 2018
@bastelfreak
Copy link
Member

Please also rebase against our latest master branch. #172 introduced some conflicts.

@bastelfreak bastelfreak added the needs-work not ready to merge just yet label Mar 17, 2018
@blupman
Copy link
Author

blupman commented Mar 19, 2018

Hi @bastelfreak

Thanks for your feedback!, I have added my work email to the profile, also rebased, i took a few try's to figure out failing spectest on arch linux.

@blupman
Copy link
Author

blupman commented Apr 4, 2018

Hi @bastelfreak

I have updated varnish_exporter to reflect the recent changes done to the master branch.

@bastelfreak bastelfreak removed needs-rebase needs-work not ready to merge just yet labels Apr 5, 2018
@bastelfreak
Copy link
Member

thanks @blupman !

@bastelfreak bastelfreak merged commit 72be4ec into voxpupuli:master Apr 5, 2018
cegeka-jenkins pushed a commit to cegeka/puppet-prometheus that referenced this pull request Aug 28, 2019
Rovanion pushed a commit to Rovanion/puppet-prometheus that referenced this pull request May 5, 2021
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