Skip to content
This repository has been archived by the owner on Aug 22, 2022. It is now read-only.

Deprecate uptimerobot in favor of uptime-prometheus #827

Closed
onedr0p opened this issue Apr 24, 2021 · 2 comments
Closed

Deprecate uptimerobot in favor of uptime-prometheus #827

onedr0p opened this issue Apr 24, 2021 · 2 comments

Comments

@onedr0p
Copy link
Member

onedr0p commented Apr 24, 2021

Details

As we did for the speedtest chart, deprecate this one in favor of the prometheus exporter.

@billimek do you have any issue with this?

@billimek
Copy link
Member

@onedr0p I suggest against this. There is a pretty significant gap (IMO) with the uptimerobot-prometheus chart. It leverages the uptimerobot_exporter solution and that will never capture an outage that is resolved within the prometheus polling interval.

Basically, the uptimerobot_exporter works by checking the uptime robot current state at each prometheus polling interval. Consider the following:

  • t0: outage occurs
  • t1: outage resolves
  • t2: uptimerobot_exporter polls current state

Using this approach we will never have a record of the outage that occurred which diminishes the value proposition of extracting data from uptimerobot to track outages.

See this issue for some details.

@onedr0p
Copy link
Member Author

onedr0p commented Apr 30, 2021

No problem, I didn't know the difference between the two charts really. Thanks for filling me in 🤠

@onedr0p onedr0p closed this as completed Apr 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants