From: luo rixin <luorixin@huawei.com>
Date: Thu, 25 May 2023 02:39:38 +0000 (+0800)
Subject: crimson/osd/ops_executer: fix double apply_stats() when writing
X-Git-Tag: v18.1.2~28^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=f8a3a667d1024c6a151333d2408ecae2df0617f9;p=ceph-ci.git

crimson/osd/ops_executer: fix double apply_stats() when writing

As double apply_stats() when writing, the write related metrics such as
the object number and space utilization are doubled. Therefore, just do once in
ops executing process.

Fixes: https://tracker.ceph.com/issues/61227

Signed-off-by: luo rixin <luorixin@huawei.com>
(cherry picked from commit 1dd18476f09130cd0eefaa7e9c35d476631709d2)
---

diff --git a/src/crimson/osd/ops_executer.h b/src/crimson/osd/ops_executer.h
index 97e0bbda856..1230b1c5a2e 100644
--- a/src/crimson/osd/ops_executer.h
+++ b/src/crimson/osd/ops_executer.h
@@ -522,7 +522,6 @@ OpsExecuter::flush_changes_n_do_ops_effects(
       txn
     ).then_interruptible([mut_func=std::move(mut_func),
                           this](auto&& log_entries) mutable {
-      apply_stats();
       auto [submitted, all_completed] =
         std::forward<MutFunc>(mut_func)(std::move(txn),
                                         std::move(obc),