Add influxdb monitoring #67
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
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! |
|
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! |
RoyWiggins
approved these changes
Feb 14, 2024
|
We finally managed to review the PR today and merged it. Thank you for contributing this!! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.