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

mgr/dashboard: fix tox test failure #29125

Merged
merged 1 commit into from
Jul 19, 2019
Merged

mgr/dashboard: fix tox test failure #29125

merged 1 commit into from
Jul 19, 2019

Conversation

bk201
Copy link
Contributor

@bk201 bk201 commented Jul 19, 2019

api/task now also returns tasks in progress module.
Mock related function for tests.

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

Signed-off-by: Kiefer Chang kiefer.chang@suse.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

`api/task` now also returns tasks in progress module.
Mock related function for tests.

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

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
@tchaikov
Copy link
Contributor

113/178 Test   #5: run-tox-mgr-dashboard ...................   Passed  173.27 sec

Copy link
Member

@callithea callithea left a comment

Choose a reason for hiding this comment

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

lgtm

@callithea callithea requested a review from rjfd July 19, 2019 08:06
Copy link
Contributor

@rjfd rjfd left a comment

Choose a reason for hiding this comment

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

lgtm

@tchaikov tchaikov merged commit 000be8e into ceph:master Jul 19, 2019
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.

4 participants