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: pool stats not returned by default #25635

Merged

Conversation

alfonsomthd
Copy link
Contributor

@alfonsomthd alfonsomthd commented Dec 19, 2018

  • All pool controller methods with same default value
    for stats flag.
  • Stats requested explicitly by frontend service.
  • Updated API tests accordingly.

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

Signed-off-by: Alfonso Martínez almartin@redhat.com

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

@alfonsomthd alfonsomthd force-pushed the 36740-pool-stats-disabled-by-default branch 2 times, most recently from cb1eb3c to d55c1b3 Compare December 20, 2018 14:49
@tchaikov
Copy link
Contributor

@alfonsomthd needs rebase

* All pool controller methods with same default value
for stats flag.
* Stats requested explicitly by frontend service.
* Updated API tests accordingly.

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

Signed-off-by: Alfonso Martínez <almartin@redhat.com>
@tchaikov
Copy link
Contributor

tchaikov commented Jan 3, 2019

guess we don't need #25761 if this PR gets merged.

edit, they fix different issues.

@tchaikov tchaikov merged commit 9a92825 into ceph:master Jan 3, 2019
@alfonsomthd alfonsomthd deleted the 36740-pool-stats-disabled-by-default branch April 4, 2019 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants