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

mds: relevel debug message levels for balancer/migrator #33471

Merged
merged 2 commits into from
Feb 25, 2020

Conversation

batrick
Copy link
Member

@batrick batrick commented Feb 21, 2020

Some of these messages are too verbose or inconsistent with other
messages.

Signed-off-by: Patrick Donnelly pdonnell@redhat.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 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

@batrick batrick added cephfs Ceph File System needs-review labels Feb 21, 2020
@batrick batrick added this to the octopus milestone Feb 21, 2020
@batrick batrick requested a review from markhpc February 21, 2020 19:10
Copy link
Member

@markhpc markhpc left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me!

src/mds/MDBalancer.cc Outdated Show resolved Hide resolved
src/mds/MDBalancer.cc Outdated Show resolved Hide resolved
Copy link
Member

@markhpc markhpc left a comment

Choose a reason for hiding this comment

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

Other than the one nit it lgtm!

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Some of these messages are too verbose or inconsistent with other
messages. Also, clean up function name printing and be consistent
for all functions.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
@batrick batrick merged commit d4891c8 into ceph:master Feb 25, 2020
@batrick batrick deleted the mds-migrator-dout branch February 25, 2020 19:02
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.

2 participants