os, test: make omap_iterate obligatory for ObjectStores

Signed-off-by: Radoslaw Zarzynski <[email protected]>

Author: rzarzynski

src/os/ObjectStore.h

@@ -798,9 +798,7 @@ class ObjectStore {
     omap_iter_seek_t start_from,
     std::function<omap_iter_ret_t(std::string_view,
                                   std::string_view)> visitor
-  ) {
-    return -EOPNOTSUPP;
-  }
+  ) = 0;
 
   virtual int flush_journal() { return -EOPNOTSUPP; }
 

src/test/objectstore/ObjectStoreImitator.h

@@ -347,6 +347,16 @@ class ObjectStoreImitator : public ObjectStore {
                     ) override {
     return {};
   }
+
+  int omap_iterate(CollectionHandle &c,   ///< [in] collection
+                   const ghobject_t &oid, ///< [in] object
+                   /// [in] where the iterator should point to at the beginning
+                   omap_iter_seek_t start_from,
+                   std::function<omap_iter_ret_t(std::string_view, std::string_view)> f
+                   ) override {
+    return 0;
+  }
+
   void set_fsid(uuid_d u) override {}
   uuid_d get_fsid() override { return {}; }
   uint64_t estimate_objects_overhead(uint64_t num_objects) override {