-
Notifications
You must be signed in to change notification settings - Fork 18
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
Added new metrics and updated gobgp version used. #35
Conversation
Use gobgp v3.23.0
CLIC-477: Update gobgp dependencies to use commit hash - 7d2d020bd226
Cherry-picked, modified commit 2815be1 adding peer metrics
Fix for advertised_prefix_count metric always equal to 0 in dashboards
@mrueg , i hope all is well! Do you still use the exporter? I am hoping to find someone to review the PR, because the last time I touched GoBGP was probably 5 years ago. |
gobgp now has prometheus support built in https://github.com/osrg/gobgp/blob/ad7dc34e102097cf338cea26b73de3f05d6a8029/internal/pkg/metrics/metrics.go might be worth adding those there. |
@mrueg , thank you for quick reply! Do you think it is worth maintaining this exporter? |
@piotrsuchy , does Akamai want to maintain this exporter? If yes, I am open to discuss. |
Ideally, missing metrics would be included in gobgp's upstream and this one wouldn't be needed anymore. |
@piotrsuchy , please see @mrueg , why don’t you use the exporter included with GoBGP? |
Hello, thanks for the involvement! We are using gobgp_exporter mostly for legacy reasons, as we were using gobgp exporter before there were standalone metrics for gobgp. If that's something you agree on we would be willing to maintain the exporter. However using gobgp directly might be the better approach - I will talk to the rest of the people involved and will come back to you with a response. |
Hello again, after some further talks with the rest of the team we decided to try to upstream our changes with additional metrics to gobgp directly. In the meantime we will still probably work on the fork of gobgp_exporter for our 'more internal' and less general stuff. Thanks for the feedback, I'm closing this PR. |
Updated gobgp version used from v3.12.0 to v3.27.0 + commits until 7d2d020bd226, as that version of gobgp has a useful bugfix.
Also added new metrics:
Our company's fork of gobgp_exporter got a bit ahead of the upstream so we would love to get this merged, and continue working on the upstream repo instead of our fork. Please give us feedback or let us know if merging is possible :)