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.

Sign up for GitHub

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

Jump to bottom

mgr/dashboard: adjust refresh intervals of Services and Daemons #38441

Merged
merged 1 commit into from Dec 10, 2020
Merged

mgr/dashboard: adjust refresh intervals of Services and Daemons #38441

merged 1 commit into from Dec 10, 2020

Conversation

bk201
Copy link
Contributor

@bk201 bk201 commented Dec 4, 2020

Change the interval from 60 seconds to 5 seconds.

Fixes: https://tracker.ceph.com/issues/48455
Signed-off-by: Kiefer Chang kiefer.chang@suse.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@bk201
mgr/dashboard: adjust refresh intervals of Services and Daemons
4bea4e9 
Change the interval from 60 seconds to 5 seconds.

Fixes: https://tracker.ceph.com/issues/48455
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
@bk201 bk201 requested a review from a team as a code owner December 4, 2020 03:36
@bk201 bk201 requested review from Exotelis, s0nea and avanthakkar and removed request for a team December 4, 2020 03:36
@github-actions github-actions bot added the pybind label Dec 4, 2020
@bk201 bk201 added this to In progress in Dashboard via automation Dec 4, 2020
@bk201 bk201 requested a review from a team December 4, 2020 03:37
Dashboard automation moved this from In progress to Reviewer approved Dec 4, 2020
@callithea
Copy link
Member

jenkins test dashboard

s0nea
s0nea approved these changes Dec 8, 2020
View reviewed changes
Copy link
Member

@s0nea s0nea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@s0nea s0nea added the skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology label Dec 8, 2020
@epuertat epuertat moved this from Reviewer approved to Ready-to-merge in Dashboard Dec 9, 2020
@epuertat epuertat merged commit 1586646 into ceph:master Dec 10, 2020
Dashboard automation moved this from Ready-to-merge to Done Dec 10, 2020
@callithea callithea mentioned this pull request Dec 15, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@s0nea s0nea s0nea approved these changes

@avanthakkar avanthakkar avanthakkar approved these changes

@callithea callithea callithea approved these changes

@nizamial09 nizamial09 nizamial09 approved these changes

@Exotelis Exotelis Awaiting requested review from Exotelis

Assignees
No one assigned
Labels
bug-fix dashboard needs-review pybind skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology
Projects
Ceph-Dashboard
Archived in project
Dashboard
  
Done
Milestone
No milestone
6 participants
@bk201 @callithea @s0nea @avanthakkar @nizamial09 @epuertat