Skip to content

Add network exporter#1647

Closed
syepes wants to merge 4 commits intoprometheus:masterfrom
syepes:add_network_exporter
Closed

Add network exporter#1647
syepes wants to merge 4 commits intoprometheus:masterfrom
syepes:add_network_exporter

Conversation

@syepes
Copy link
Contributor

@syepes syepes commented May 27, 2020

Addition of the Network Exporter

syepes added 4 commits May 27, 2020 21:18
Signed-off-by: Sebastian YEPES <syepes@gmail.com>
Signed-off-by: Sebastian YEPES <syepes@gmail.com>
Signed-off-by: Sebastian YEPES <syepes@gmail.com>
Signed-off-by: Sebastian YEPES <syepes@gmail.com>
@syepes syepes force-pushed the add_network_exporter branch from f8d666c to 196e84e Compare May 27, 2020 19:18
@brian-brazil
Copy link
Contributor

Thanks for your PR. I aim to avoid duplication in this list, and this network exporter seems to duplicate both the blackbox and especially smokeping exporters which are already listed. If there's ways in which you feel those could be improved, I'd encourage you to talk to their maintainers.

@brian-brazil brian-brazil added the exporters and integrations Requests for new entries in the list of exporters and integrations label May 27, 2020
@syepes
Copy link
Contributor Author

syepes commented May 27, 2020

@brian-brazil Actually is a mix of both blackbox and smokeping but with more metrics (jitter stats) and features but specially the MTR module (traceroute)

@brian-brazil
Copy link
Contributor

At the end of the day it's filling the same base use cases that already listed exporters do, so I don't think it makes sense to list this here. You are free to announce your exporter on the -users list.

@syepes
Copy link
Contributor Author

syepes commented May 27, 2020

@brian-brazil, Ok no problem

@syepes syepes closed this May 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exporters and integrations Requests for new entries in the list of exporters and integrations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants