LU-10633 mdt: Convert MDS restoring RPC message to D_WARNING

Using D_WARNING instead of D_RPCTRACE causes the message to be both
logged in the Lustre DK logs and on the system console.  This patch
changes the MDS restore/replay debug message to use D_WARNING.

A restored/replayed metadata request indicates some sort of underlying
error, and even when handled correctly, should generate a warning.

Test-Parameters: trivial
Change-Id: Iff98521853323469fc5d6c7d546ca83477b1cb9f
HPE-bug-id: LUS-2578
Signed-off-by: Chris Horn <chris.horn@hpe.com>
Reviewed-on: https://review.whamcloud.com/31214
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Vladimir Saveliev <vlaidimir.saveliev@hpe.com>
Reviewed-by: Vitaly Fertman <vitaly.fertman@hpe.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>

diff --git a/lustre/mdt/mdt_recovery.c b/lustre/mdt/mdt_recovery.c
index 22e692d..45d404e 100644 (file)
--- a/lustre/mdt/mdt_recovery.c
+++ b/lustre/mdt/mdt_recovery.c
@@ -185,8 +185,7 @@ __u64 mdt_req_from_lrd(struct ptlrpc_request *req,
        LASSERT(trd != NULL);
        lrd = &trd->trd_reply;
 
-       DEBUG_REQ(D_HA, req, "restoring transno %lld/status %d",
-                 lrd->lrd_transno, lrd->lrd_result);
+       DEBUG_REQ(D_HA, req, "restoring transno");
 
        req->rq_transno = lrd->lrd_transno;
        req->rq_status = lrd->lrd_result;
@@ -198,8 +197,7 @@ __u64 mdt_req_from_lrd(struct ptlrpc_request *req,
        lustre_msg_set_transno(req->rq_repmsg, req->rq_transno);
        lustre_msg_set_status(req->rq_repmsg, req->rq_status);
 
-       DEBUG_REQ(D_RPCTRACE, req, "restoring transno %lld/status %d",
-                 req->rq_transno, req->rq_status);
+       DEBUG_REQ(D_WARNING, req, "restoring transno");
 
        mdt_steal_ack_locks(req);