cmd/ocprometheus: Set timestamps

aristanetworks / goarista

Fairly general building blocks used in Arista Go code and open-sourced for the benefit of all.

Apache License 2.0

206 stars 66 forks source link

cmd/ocprometheus: Set timestamps #64

Open RavuAlHemio opened 2 years ago

RavuAlHemio commented 2 years ago

Now that Prometheus and OpenMetrics support timestamps, forward them from gNMI to the exporter.

RavuAlHemio commented 2 years ago

Okay, this might not have the desired effect; apparently Prometheus only stores the value with the given timestamp and assumes that the value is no longer valid if the timestamp is not updated during the next scrape.

Perhaps the discussion of timestamps should simply be removed from the README (or replaced with an explanation why ocprometheus does not consume them, as above)?