Hi, Any chance someone could take a look at this, we have written a prometheus exporter for bgp and ospf, which we would like to share, but it relies on having this json support in place.
I have re-attached the original diff. Thanks Richard On Mon, Jun 1, 2020 at 7:52 AM Richard Chivers <r.chiv...@zengenti.com> wrote: > Hi, > > Just a quick bump, wondering if someone could take a look at the diff, > conscious there appears to have been a lot going on recently! > > I want to do some more detailed work on the json output, but would > ideally get this first iteration in first. > > Richard > > On Sat, May 23, 2020 at 9:28 AM Richard Chivers <r.chiv...@zengenti.com> > wrote: > > > > Hi, > > > > I have attached the first iteration of the ospf json support. Sorry > > about the large commit, > > but it had to come in one go, if we didn't want a broken implementation. > > > > I will go back and update output.c and output_json.c to remove the > detail flag > > and instead pass through the cli parse_result as is done in bgpctl -j sh > rib, > > but thought it made sense to get the main commit out of the way and > > then circle back > > to these tweaks next week. > > > > In general I think there should be a consideration over how times are > output. > > > > At present this diff outputs time using the pretty standard approach > > used in ospfctl all along, > > which is also what bgpctl -j sh rib does. > > > > I question whether for durations and uptime we should perhaps > > introduce something like > > > > uptime_seconds: 30 > > uptime_seconds: null > > > > as well as or rather than: > > > > uptime: "6d06h31m" > > uptime: "00:00:00" > > > > I'm not sure if the above is a standard format (6d06h31m) that can be > parsed > > easily in many languages, which is part of what json support adds to the > mix, > > if it is then it is the best of both worlds already! > > > > I can take a dig into this, but i'm sure someone will know off hand, > > if so what is the RFC or equiv? > > > > The only aspects I have not tested are: > > > > ospfctl -j show database summary > > ospfctl -j show database opaque > > > > These have no results in my configuration. > > > > I do question wheter ospfctl -j show database summary just needs > removing? > > I will perhaps have a dig into ospfd and see what generates the relevant > imsg. >
ospfctl_json.diff
Description: Binary data