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
Build HAProxy Package with buildin Prometheus exporter. Implement #10500 #861
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to me this should be added as a configurable option for the FreeBSD port and that should be submitted upstream to FreeBSD first.
it is already there FreeBSD-ports/net/haproxy/Makefile Line 39 in a198c1a
PR updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If there is already an option for it, then this PR is unnecessary for this repo. See my note inline.
net/haproxy/Makefile
Outdated
@@ -28,7 +28,7 @@ DPCRE_DESC= Link dynamically | |||
SPCRE_DESC= Link statically | |||
DEVICEATLAS_DESC= DeviceAtlas Device Detection support | |||
PROMEX_DESC= Enable Prometheus exporter | |||
OPTIONS_DEFAULT= SPCRE OPENSSL | |||
OPTIONS_DEFAULT= SPCRE OPENSSL PROMEX |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't change the default options. Add/change the relevant lines in the pfSense repository in https://github.com/pfsense/pfsense/blob/master/tools/conf/pfPorts/make.conf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done: pfsense/pfsense#4320
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We just need to make sure this gets merged after the make.conf change in #4320
Redmine Issue: https://redmine.pfsense.org/issues/10500
Ready for review
However, PROMEX is not built by default with HAProxy. It is provided as an extra
component for everyone want to use it. So you need to explicitly build HAProxy
with the PROMEX service, using the Makefile variable "EXTRA_OBJS". For instance:
make TARGET=linux-glibc EXTRA_OBJS="contrib/prometheus-exporter/service-prometheus.o"
if HAProxy provides the PROMEX service, the following build option will be
reported by the command "haproxy -vv":
https://github.com/haproxy/haproxy/blob/master/contrib/prometheus-exporter/README
needs testing on 2.5 first