Add whether or not to allocate the integrity superblock

This is an option in the D-Bus, the predict script and the pool-level
metadata.

Signed-off-by: mulhern <[email protected]>

Author: mulkieran

src/bin/utils/cmds.rs

@@ -110,6 +110,12 @@ pool is encrypted, setting this option has no effect on the prediction."),
                         .default_value(Box::leak((*DEFAULT_INTEGRITY_JOURNAL_SIZE).to_string().into_boxed_str()) as &'static str)
                         .help("Size of the integrity journal. Default is 128 MiB. Units are bytes.")
                         .next_line_help(true)
+                    )
+                    .arg(
+                        Arg::new("no_integrity_superblock")
+                        .long("no-integrity-superblock")
+                        .help("Do not allocate space for integrity superblock")
+                        .next_line_help(true)
                     ),
                 Command::new("filesystem")
                     .about("Predicts the space usage when creating a Stratis filesystem.")
@@ -166,6 +172,7 @@ impl<'a> UtilCommand<'a> for StratisPredictUsage {
                             })
                             .expect("default specified by parser"),
                     ),
+                    allocate_superblock: Some(!sub_m.get_flag("no_integrity_superblock")),
                 })?,
                 LevelFilter::from_str(
                     matches

src/dbus_api/api/manager_3_8/api.rs

@@ -68,6 +68,13 @@ pub fn create_pool_method(f: &Factory<MTSync<TData>, TData>) -> Method<MTSync<TD
         //
         // Rust representation: (bool, String)
         .in_arg(("tag_spec", "(bs)"))
+        // Optionally specify whether to reserve space for integrity
+        // superblock.
+        // b: true if the second value is to be read, otherwise false.
+        // b: true if the superblock reservation is supposed to be done
+        //
+        // Rust representation: (bool, bool)
+        .in_arg(("allocate_superblock", "(bb)"))
         // In order from left to right:
         // b: true if a pool was created and object paths were returned
         // o: Object path for Pool

src/dbus_api/api/manager_3_8/methods.rs

@@ -159,6 +159,7 @@ pub fn create_pool(m: &MethodInfo<'_, MTSync<TData>, TData>) -> MethodResult {
     );
     let journal_size_tuple: (bool, u64) = get_next_arg(&mut iter, 4)?;
     let tag_spec_tuple: (bool, String) = get_next_arg(&mut iter, 5)?;
+    let allocate_superblock_tuple: (bool, bool) = get_next_arg(&mut iter, 6)?;
 
     let return_message = message.method_return();
 
@@ -201,14 +202,17 @@ pub fn create_pool(m: &MethodInfo<'_, MTSync<TData>, TData>) -> MethodResult {
         }
     };
 
+    let allocate_superblock = tuple_to_option(allocate_superblock_tuple);
+
     let dbus_context = m.tree.get_data();
     let create_result = handle_action!(block_on(dbus_context.engine.create_pool(
         name,
         &devs.map(Path::new).collect::<Vec<&Path>>(),
         EncryptionInfo::from_options((key_desc, clevis_info)).as_ref(),
         IntegritySpec {
             journal_size,
-            tag_spec
+            tag_spec,
+            allocate_superblock,
         },
     )));
     match create_result {

src/engine/types/mod.rs

@@ -527,13 +527,15 @@ impl IntegrityTagSpec {
 pub struct IntegritySpec {
     pub tag_spec: Option<IntegrityTagSpec>,
     pub journal_size: Option<Bytes>,
+    pub allocate_superblock: Option<bool>,
 }
 
 #[derive(Clone, Copy, Debug, Deserialize, Eq, PartialEq, Serialize)]
 pub struct ValidatedIntegritySpec {
     pub tag_spec: IntegrityTagSpec,
     pub journal_size: Sectors,
     pub block_size: Bytes,
+    pub allocate_superblock: bool,
 }
 
 impl Default for ValidatedIntegritySpec {
@@ -563,6 +565,7 @@ impl TryFrom<IntegritySpec> for ValidatedIntegritySpec {
             journal_size,
             tag_spec: spec.tag_spec.unwrap_or(DEFAULT_INTEGRITY_TAG_SPEC),
             block_size: DEFAULT_INTEGRITY_BLOCK_SIZE,
+            allocate_superblock: spec.allocate_superblock.unwrap_or(true),
         })
     }
 }

tests/client-dbus/src/stratisd_client_dbus/_introspect.py

@@ -15,6 +15,7 @@
       <arg name="clevis_info" type="(b(ss))" direction="in" />
       <arg name="journal_size" type="(bt)" direction="in" />
       <arg name="tag_spec" type="(bs)" direction="in" />
+      <arg name="allocate_superblock" type="(bb)" direction="in" />
       <arg name="result" type="(b(oao))" direction="out" />
       <arg name="return_code" type="q" direction="out" />
       <arg name="return_string" type="s" direction="out" />

tests/client-dbus/tests/udev/_utils.py

@@ -152,6 +152,7 @@ def create_v2_pool():
                 ),
                 "journal_size": (False, 0),
                 "tag_spec": (False, ""),
+                "allocate_superblock": (False, False),
             },
         )