Merge pull request ceph#57836 from guojidan/warn-pool-app

src/mon/PGMap: raise POOL_APP_NOT_ENABLED after some time for new pool

Reviewed-by: Ilya Dryomov <[email protected]>

Author: idryomov

src/common/options/mgr.yaml.in

@@ -292,6 +292,15 @@ options:
   default: true
   services:
   - mgr
+- name: mon_warn_on_pool_no_app_grace
+  type: secs
+  level: dev
+  desc: time after which POOL_APP_NOT_ENABLED health warning is issued
+  default: 5_min
+  services:
+  - mgr
+  see_also:
+  - mon_warn_on_pool_no_app
 - name: mon_warn_on_too_few_osds
   type: bool
   level: advanced

src/mon/PGMap.cc

@@ -3352,9 +3352,13 @@ void PGMap::get_health_checks(
       // application metadata is not encoded until luminous is minimum
       // required release
       if (pool.application_metadata.empty() && !pool.is_tier()) {
-        stringstream ss;
-        ss << "application not enabled on pool '" << pool_name << "'";
-        detail.push_back(ss.str());
+        utime_t now(ceph::real_clock::now());
+        if ((now - pool.get_create_time()) >
+            g_conf().get_val<std::chrono::seconds>("mon_warn_on_pool_no_app_grace").count()) {
+          stringstream ss;
+          ss << "application not enabled on pool '" << pool_name << "'";
+          detail.push_back(ss.str());
+        }
       }
     }
     if (!detail.empty()) {