Revert unnecessary change

stratis-storage · Jun 17, 2022 · 7675d53 · 7675d53
1 parent 6f7c816
commit 7675d53
Showing 1 changed file with 19 additions and 15 deletions.
diff --git a/src/engine/strat_engine/thinpool/thinpool.rs b/src/engine/strat_engine/thinpool/thinpool.rs
@@ -622,21 +622,25 @@ impl ThinPool {
         let mdv = MetadataVol::setup(pool_uuid, mdv_dev)?;
         let filesystem_metadatas = mdv.filesystems()?;
 
-        let mut filesystems = Vec::with_capacity(filesystem_metadatas.len());
-        for fssave in filesystem_metadatas.iter() {
-            match StratFilesystem::setup(pool_uuid, &thinpool_dev, fssave) {
-                Ok(fs) => {
-                    fs.udev_fs_change(pool_name, fssave.uuid, &fssave.name);
-                    filesystems.push((Name::new(fssave.name.to_owned()), fssave.uuid, fs));
-                }
-                Err(err) => {
-                    warn!(
-                        "Filesystem specified by metadata {:?} could not be setup, reason: {:?}",
-                        fssave, err
-                    );
-                }
-            }
-        }
+        let filesystems = filesystem_metadatas
+            .iter()
+            .filter_map(
+                |fssave| match StratFilesystem::setup(pool_uuid, &thinpool_dev, fssave) {
+                    Ok(fs) => {
+                        fs.udev_fs_change(pool_name, fssave.uuid, &fssave.name);
+                        Some((Name::new(fssave.name.to_owned()), fssave.uuid, fs))
+                    },
+                    Err(err) => {
+                        warn!(
+                            "Filesystem specified by metadata {:?} could not be setup, reason: {:?}",
+                            fssave,
+                            err
+                        );
+                        None
+                    }
+                },
+            )
+            .collect::<Vec<_>>();
 
         let mut fs_table = Table::default();
         for (name, uuid, fs) in filesystems {