Merge pull request #20121 from batrick/i21252p2

luminous: mds: fix return value of MDCache::dump_cache

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

diff --cc src/mds/MDCache.cc
index 6df775b5a340bb8a61c071e0126fb99b48c9d122,c4677c80efac2597f3a46d26802b150255272329..298799f0d9529688cb3164a69bd0f270f732abf5
--- 1/src/mds/MDCache.cc
--- 2/src/mds/MDCache.cc
+++ b/src/mds/MDCache.cc
@@@ -12003,19 -12003,8 +12003,20 @@@ int MDCache::dump_cache(const char *fn
      if (f) {
        f->close_section();  // inode
      }
 +    return 1;
 +  };
 +
 +  for (auto p : inode_map) {
 +    r = dump_func(p.second);
 +    if (r < 0)
 +      goto out;
 +  }
 +  for (auto p : snap_inode_map) {
 +    r = dump_func(p.second);
 +    if (r < 0)
 +      goto out;
    }
+   r = 0;
  
   out:
    if (f) {