mds: don't respond getattr with -EROFS when mds is readonly

Fixes: https://tracker.ceph.com/issues/43601
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 35da262d408a17aa4c4f3a938eadb274b51c98e7..da11f9d7d8622e47820c690544254cc8fe176b61 100644 (file)
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -2435,12 +2435,16 @@ void Server::dispatch_client_request(MDRequestRef& mdr)
 
   dout(7) << "dispatch_client_request " << *req << dendl;
 
-  if (mdcache->is_readonly() ||
-      (mdr->has_more() && mdr->more()->slave_error == -EROFS)) {
+  if (req->may_write() && mdcache->is_readonly()) {
     dout(10) << " read-only FS" << dendl;
     respond_to_request(mdr, -EROFS);
     return;
   }
+  if (mdr->has_more() && mdr->more()->slave_error) {
+    dout(10) << " got error from slaves" << dendl;
+    respond_to_request(mdr, mdr->more()->slave_error);
+    return;
+  }
   
   if (is_full) {
     if (req->get_op() == CEPH_MDS_OP_SETLAYOUT ||