vbezgachev / maxctrl_exporter

MaxScale metrics exporter for Prometheus
Apache License 2.0
12 stars 11 forks source link

contributing to project #4

Closed rvlane closed 2 years ago

rvlane commented 3 years ago

I work for Nokia and we are moving to maxscale-2.5. Therefore we need an exporter that uses the 8989 maxctrl interface, thus we were looking at this project. As we are testing we have a couple of major requirements:

  1. maxscale passwords must not be exposed via any environment variable or text files. We are still working how we would accomplish this.
  2. We are required to use TLS for all interfaces, including (a) the maxctrl 8989 port (maxscale is configured with TLS for the maxctrl port) and (b) the scraping port (for us 9195).

The older maxscale exporter we were using does not support https either, but we need to have this integrated. So are you open to additional contributors to this project to implement capabilities that will meet out needs? I need to get it cleared through the Nokia process, however wanted to check here first.

vbezgachev commented 3 years ago

Hi @rvlane .Me and @ktugan will be happy if you contribute to the project. Feel free to makes the changes and open a PR. Currently, I just don't have time to implement feature requests :-(

pgporada commented 2 years ago

@rvlane This exporter is now capable of scraping the maxscale rest api over TLS as of https://github.com/Vetal1977/maxctrl_exporter/pull/7. We've been using this in our maxscale 6.x environment since. You can adjust the exporter port via environment variables. Here's our system unit file.

[Unit]
Description=Prometheus exporter for MaxScale server metrics.
Documentation=https://github.com/Vetal1977/maxctrl_exporter
After=network.target

[Service]
Type=simple
Environment=MAXSCALE_USERNAME="viewonlyuser"
Environment=MAXSCALE_PASSWORD="SECRET"
Environment=MAXSCALE_URL="https://fqdn:8989"
Environment=MAXSCALE_EXPORTER_PORT=8093
Environment=MAXSCALE_CA_CERTIFICATE="/etc/someproduct/ca.cert"
ExecStart=/bin/maxctrl_exporter

[Install]
WantedBy=multi-user.target
vbezgachev commented 2 years ago

@rvlane I hope, you wouldn't mind if I close the issue