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: deletion of host doesn't reflect the dashboard #38186

Closed
wants to merge 1 commit into from
Closed

mgr/dashboard: deletion of host doesn't reflect the dashboard #38186

wants to merge 1 commit into from

Conversation

avanthakkar
Copy link
Contributor

@avanthakkar avanthakkar commented Nov 19, 2020 •
edited

This PR fixes the updation of the frontend host list upon the deletion of hosts

Fixes: https://tracker.ceph.com/issues/48290

Signed-off-by: Avan Thakkar athakkar@redhat.com

Deleted ceph-node-01/02

Before:
Screenshot from 2020-11-19 12-57-20

After:
Screenshot from 2020-11-19 18-05-28

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

@avanthakkar avanthakkar added bug-fix dashboard skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology labels Nov 19, 2020
@avanthakkar avanthakkar requested review from alfonsomthd and a team November 19, 2020 13:36
@avanthakkar avanthakkar self-assigned this Nov 19, 2020
@avanthakkar avanthakkar added this to In progress in Dashboard via automation Nov 19, 2020
@avanthakkar avanthakkar requested review from Exotelis, callithea, a team, prokopto-dev and epuertat and removed request for a team, Exotelis and callithea November 19, 2020 13:36
@callithea
Copy link
Member

jenkins retest this please

@epuertat epuertat closed this Dec 14, 2020
Dashboard automation moved this from In progress to Done Dec 14, 2020
@epuertat
Copy link
Member

Closed as this behaviour should be implemented by cephadm.

@epuertat epuertat deleted the fix-frontend-host-list branch December 14, 2020 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alfonsomthd alfonsomthd Awaiting requested review from alfonsomthd

@Exotelis Exotelis Awaiting requested review from Exotelis

@prokopto-dev prokopto-dev Awaiting requested review from prokopto-dev

@epuertat epuertat Awaiting requested review from epuertat

Assignees

@avanthakkar avanthakkar

Labels
bug-fix dashboard needs-review 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
3 participants
@avanthakkar @callithea @epuertat