search

mercure-imaging / mercure

mercure DICOM Orchestrator
https://mercure-imaging.org
MIT License
66 stars 32 forks source link

Add influxdb monitoring #67

Closed aarontovars closed 9 months ago

aarontovars commented 11 months ago

This PR introduces InfluxDB support to our monitoring stack, alongside Graphite.

The key driver for this addition is the need for more secure communication protocols in our monitoring tools. Unlike Graphite, which is limited to UDP/TCP, InfluxDB supports HTTPS, offering enhanced security. It also brings token-based authentication and the ability to specify organization and data buckets, aligning better with security standards and modern monitoring needs.

aarontovars commented 11 months ago

Hi @tblock79 @RoyWiggins @joshy ,

Did you have a chance to take a look at this PR? If you have any doubt or any clarification, please do not hesitate to contact me!

rogerbramon commented 9 months ago

Hi @tblock79 and @RoyWiggins,

We would like to hear your opinion on this PR.

We believe that adding an additional option to Graphite could enhance the project, particularly for storing logs outside the intranet. This would allow for a secure connection using HTTPS.

If you have any alternative suggestions or find this inconvenient, we are open to discussing it. We would prefer to contribute to the project rather than deviate from it to fulfill our requirements.

Thank you!

tblock79 commented 9 months ago

We finally managed to review the PR today and merged it. Thank you for contributing this!!