From 548af10f1057fe3fb46e23887b012a664b45b7f8 Mon Sep 17 00:00:00 2001 From: Dhairya Parmar Date: Mon, 22 May 2023 16:07:34 +0530 Subject: [PATCH] mds: remove code to bypass dumping empty header scrub info Previously when ~mdsdir was scrubbed at CephFS root, it's header was kept empty, thus it became necessary to not dump it's values for 'scrub status'. Now since both the scrubs(~mdsdir and root) run under the same header, this code is no more needed. Fixes: https://tracker.ceph.com/issues/59350 Signed-off-by: Dhairya Parmar --- src/mds/ScrubStack.cc | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/mds/ScrubStack.cc b/src/mds/ScrubStack.cc index a544af7665e69..6d799343f1496 100644 --- a/src/mds/ScrubStack.cc +++ b/src/mds/ScrubStack.cc @@ -668,11 +668,6 @@ void ScrubStack::scrub_status(Formatter *f) { have_more = false; auto& header = p.second; - if (mdcache->get_inode(header->get_origin())->is_mdsdir() - && header->get_scrub_mdsdir() && header->get_tag().empty()) { - continue; - } - std::string tag(header->get_tag()); f->open_object_section(tag.c_str()); // scrub id -- 2.39.5