Skip to content
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

PAUSED Sensor will not been shown in /metrics page #3579

Closed
2 tasks done
RK62 opened this issue Aug 15, 2023 · 5 comments
Closed
2 tasks done

PAUSED Sensor will not been shown in /metrics page #3579

RK62 opened this issue Aug 15, 2023 · 5 comments
Labels
area:metrics related to monitoring metrics feature-request Request for new features to be added

Comments

@RK62
Copy link

RK62 commented Aug 15, 2023

⚠️ Please verify that this bug has NOT been raised before.

  • I checked and didn't find similar issue

🛡️ Security Policy

Description

Sensors that are on PAUSE are not listed on the /metrics page under "monitor_status".
States MAINTENANCE, UP and DOWN are always complete.
The sensor type does not matter.
The number on the "QuickStats" page is also correct.

If you set a group to pause, QuickStats will show the correct number including the underlying sensors, but the /metrics page will show them as UP.

👟 Reproduction steps

Set sensors to State PAUSE and call /metrics page
The call of the /metric page was authorised by API-KEY.

👀 Expected behavior

The State / Counter of QuickStats und /Metrices page should be the same.

😓 Actual Behavior

The Information of sensors and /metric page differs.

🐻 Uptime-Kuma Version

1.22.1

💻 Operating System and Arch

Docker: louislam/uptime-kuma:1.22.1

🌐 Browser

Edge

🐋 Docker Version

20.10.25

🟩 NodeJS Version

No response

📝 Relevant log output

No response

@RK62 RK62 added the bug Something isn't working label Aug 15, 2023
@chakflying
Copy link
Collaborator

I believe it's by design, metrics is not supposed to show stale data.

https://serverfault.com/questions/1005300/do-prometheus-metrics-have-some-sort-of-freshness-lifetime

@RK62
Copy link
Author

RK62 commented Sep 3, 2023

After prolonged observation, this is probably not a mistake:

  1. PENDING describes a coming error and not paused sensors. PAUSED sensors are not in the output.

  2. UP sensors with type PUSH will only be shown when a push message has been received for the first time after a restart/reboot of Uptime Kuma.

It would be very useful to be able to retrieve a JSON with all current monitor values.
Please switch to "feature-request".

@louislam louislam added feature-request Request for new features to be added and removed bug Something isn't working labels Sep 3, 2023
@CommanderStorm
Copy link
Collaborator

We already have the following APIs. Do they solve your usecase?

  • https://status.yourdomain.xyz/api/status-page/your-status-page-name
  • https://status.yourdomain.xyz/api/status-page/heartbeat/your-status-page-name

@CommanderStorm
Copy link
Collaborator

@RK62
Also note that depending on what you are requesting (your feature-request is not clear) this is likely a duplicate of #1928
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@RK62
Copy link
Author

RK62 commented Sep 3, 2023

Continued in #1928

@RK62 RK62 closed this as completed Sep 3, 2023
@CommanderStorm CommanderStorm added the area:metrics related to monitoring metrics label Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:metrics related to monitoring metrics feature-request Request for new features to be added
Projects
None yet
Development

No branches or pull requests

4 participants