Labels Milestones

Add Overall Status and Uptime to DashboardHome #437

Open

lucasra1 wants to merge 1 commits from lucasra1/overall_status into master

pull from: lucasra1/overall_status

merge into: topaLE:master

topaLE:1.6.X

topaLE:1.7.X

topaLE:1.9.X

topaLE:Mikhail5555/feature/remote-header-auth

topaLE:Nuckerr/master

topaLE:Saibamen/fix_871

topaLE:WillianRod/feat/add-favicon-badges

topaLE:andreasbrett/logging

topaLE:andreasbrett/securepush

topaLE:bertyhell/bugfix/heartbeat-bar-animation

topaLE:bertyhell/feature/monitor-checks

topaLE:bertyhell/feature/translations-extraction-script

topaLE:cert-notification

topaLE:chakflying/settings-redesign

topaLE:debian-docker

topaLE:deefdragon/Template-Engine

topaLE:deefdragon/notif-tests

topaLE:e2e-test

topaLE:fdcastel/push-api-tags

topaLE:free-disk-space

topaLE:ivanbratovic/http-basicauth

topaLE:ivanbratovic/improve-translatables

topaLE:k8s-unofficial

topaLE:master

topaLE:mhkarimi1383/master

topaLE:mrphuongbn/master

topaLE:no-need-build

topaLE:philippdormann/feature/release-management

topaLE:proffalken/feature/680_add_labels_to_prometheus_metrics

topaLE:proffalken/feature/auto_build_and_release

topaLE:rebasesoftware/feature/request-with-http-proxy

topaLE:restructure-status-page

topaLE:sqlite-upgrade-prebuilt

topaLE:tarun7singh/master

topaLE:thomasleveil/feature/565-duplicate-monitor

topaLE:thomasleveil/ux/add-group-at-the-top

Conversation 4 Commits 1 Files Changed 22

lucasra1 commented 4 years ago (Migrated from github.com)

Owner

• expand sended stats with sendAverageUptime(duration)
• Display 24h, 7d, 30d, 90d average uptime of all monitors
• Change Uptime.vue component to support displaying average uptime of all monitors

Issue #312

- expand sended stats with sendAverageUptime(duration) - Display 24h, 7d, 30d, 90d average uptime of all monitors - Change Uptime.vue component to support displaying average uptime of all monitors Issue #312

👍 1

lucasra1 (Migrated from github.com) reviewed 4 years ago

server/model/monitor.js

lucasra1 (Migrated from github.com) commented 4 years ago

Poster

Owner

I removed it, as it seems like something, someone has forgotten to remove

I removed it, as it seems like something, someone has forgotten to remove

louislam commented 4 years ago (Migrated from github.com)

Owner

Thanks, I was not implemented this because I am worry about the performance. Especially for 90d, the sql sum up a lot of data.

So I think the performance proof should be required for this pr.

Thanks, I was not implemented this because I am worry about the performance. Especially for 90d, the sql sum up a lot of data. So I think the performance proof should be required for this pr.

gaby commented 4 years ago (Migrated from github.com)

Owner

This PR will affect the performance of our uptime-kuma instance. We currently run the service with 2vCPU/2GB RAM, and we monitor close to 300 services/sites/servers.

This PR will affect the performance of our `uptime-kuma` instance. We currently run the service with 2vCPU/2GB RAM, and we monitor close to 300 services/sites/servers.

salitcao commented 4 years ago (Migrated from github.com)

Owner

I want to display average uptime 90 days. So how can I do that ?

I want to display average uptime 90 days. So how can I do that ?

deefdragon commented 4 years ago (Migrated from github.com)

Owner

A note on the performance/implementation: it might be possible to use a sliding summation/data collection for the graphs. In that case, you only need to add/remove the values that are now/are no longer relevant. The first request will be slow as the data is collected, but after that, it will be much faster.

A note on the performance/implementation: it _might_ be possible to use a sliding summation/data collection for the graphs. In that case, you only need to add/remove the values that are now/are no longer relevant. The first request will be slow as the data is collected, but after that, it will be much faster.

👍 2

This pull request has changes conflicting with the target branch.

src/languages/da-DK.js

src/languages/es-ES.js

src/languages/et-EE.js

src/languages/it-IT.js

src/languages/ja.js

src/languages/ko-KR.js

server/model/monitor.js

src/languages/de-DE.js

src/languages/en.js

src/languages/fr-FR.js

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

bug
Something isn't working dependencies
Pull requests that update a dependency file discussion doc
Improvements or additions to documentation duplicate
This issue or pull request already exists feature-request
New feature or request good first issue
Good for newcomers hacktoberfest hacktoberfest-accepted help help wanted
Extra attention is needed High
High Priority impossible invalid
This doesn't seem right investigating k8s Low
Low Priority Medium
Medium Priority News prerelease bug question
Further information is requested resolved Unknown wontfix
This will not be worked on

No Label bug dependencies discussion doc duplicate feature-request good first issue hacktoberfest hacktoberfest-accepted help help wanted High impossible invalid investigating k8s Low Medium News prerelease bug question resolved Unknown wontfix

Milestone

Set milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Assign users

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.