Skip to content

mgr/dashboard: wait longer for health status to be cleared#36320

Merged
votdev merged 1 commit intoceph:masterfrom
s0nea:wip-dashboard-46573
Jul 29, 2020
Merged

mgr/dashboard: wait longer for health status to be cleared#36320
votdev merged 1 commit intoceph:masterfrom
s0nea:wip-dashboard-46573

Conversation

@s0nea
Copy link
Member

@s0nea s0nea commented Jul 28, 2020

Because of reasons the cluster needs more time to recover from HEALTH_WARN while changes are made by test_pool_update_metadata. Lets wait several times for the cluster status to be HEALTH_OK again.

Fixes: https://tracker.ceph.com/issues/46573
Signed-off-by: Tatjana Dehler tdehler@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 dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

Because of reasons the cluster needs more time to recover from
HEALTH_WARN while changes are made by `test_pool_update_metadata`.
Lets wait several times for the cluster status to be HEALTH_OK
again.

Fixes: https://tracker.ceph.com/issues/46573
Signed-off-by: Tatjana Dehler <tdehler@suse.com>
@s0nea s0nea force-pushed the wip-dashboard-46573 branch from fea5c48 to 739b365 Compare July 28, 2020 12:49
@s0nea s0nea marked this pull request as ready for review July 28, 2020 13:08
@s0nea s0nea requested review from a team, alfonsomthd, sebastian-philipp and tchaikov July 28, 2020 13:08
@alfonsomthd alfonsomthd requested review from bk201 and votdev July 28, 2020 14:25
@alfonsomthd
Copy link
Contributor

jenkins test dashboard

@s0nea
Copy link
Member Author

s0nea commented Jul 29, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants