Merge pull request #3483 from jbaublitz/r7

Update to revision r7

.github/workflows/main.yml

@@ -281,7 +281,7 @@ jobs:
       matrix:
         include:
           - rev: HEAD
-            args: --log-level=debug stratisd_cert --verify-devices --monitor-dbus --highest-revision-number=6
+            args: --log-level=debug stratisd_cert --verify-devices --monitor-dbus --highest-revision-number=7
           - rev: v3.5.2
             args: stratisd_cert --verify-devices
     runs-on: ubuntu-22.04

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "stratisd"
-version = "3.6.0"
+version = "3.7.0"
 authors = ["Stratis Developers <[email protected]>"]
 edition = "2021"
 rust-version = "1.71.1"  # LOWEST SUPPORTED RUST TOOLCHAIN

src/dbus_api/api/mod.rs

@@ -135,6 +135,20 @@ pub fn get_base_tree<'a>(
                 .add_p(manager_3_0::version_property(&f))
                 .add_p(manager_3_2::stopped_pools_property(&f)),
         )
+        .add(
+            f.interface(consts::MANAGER_INTERFACE_NAME_3_7, ())
+                .add_m(manager_3_5::create_pool_method(&f))
+                .add_m(manager_3_0::set_key_method(&f))
+                .add_m(manager_3_0::unset_key_method(&f))
+                .add_m(manager_3_0::list_keys_method(&f))
+                .add_m(manager_3_0::destroy_pool_method(&f))
+                .add_m(manager_3_0::engine_state_report_method(&f))
+                .add_m(manager_3_4::start_pool_method(&f))
+                .add_m(manager_3_6::stop_pool_method(&f))
+                .add_m(manager_3_2::refresh_state_method(&f))
+                .add_p(manager_3_0::version_property(&f))
+                .add_p(manager_3_2::stopped_pools_property(&f)),
+        )
         .add(
             f.interface(consts::REPORT_INTERFACE_NAME_3_0, ())
                 .add_m(report_3_0::get_report_method(&f)),
@@ -162,6 +176,10 @@ pub fn get_base_tree<'a>(
         .add(
             f.interface(consts::REPORT_INTERFACE_NAME_3_6, ())
                 .add_m(report_3_0::get_report_method(&f)),
+        )
+        .add(
+            f.interface(consts::REPORT_INTERFACE_NAME_3_7, ())
+                .add_m(report_3_0::get_report_method(&f)),
         );
 
     let path = obj_path.get_name().to_owned();

src/dbus_api/blockdev/mod.rs

@@ -125,6 +125,19 @@ pub fn create_dbus_blockdev<'a>(
                 .add_p(blockdev_3_0::physical_path_property(&f))
                 .add_p(blockdev_3_0::size_property(&f))
                 .add_p(blockdev_3_3::new_size_property(&f)),
+        )
+        .add(
+            f.interface(consts::BLOCKDEV_INTERFACE_NAME_3_7, ())
+                .add_p(blockdev_3_0::devnode_property(&f))
+                .add_p(blockdev_3_0::hardware_info_property(&f))
+                .add_p(blockdev_3_0::initialization_time_property(&f))
+                .add_p(blockdev_3_0::pool_property(&f))
+                .add_p(blockdev_3_0::tier_property(&f))
+                .add_p(blockdev_3_3::user_info_property(&f))
+                .add_p(blockdev_3_0::uuid_property(&f))
+                .add_p(blockdev_3_0::physical_path_property(&f))
+                .add_p(blockdev_3_0::size_property(&f))
+                .add_p(blockdev_3_3::new_size_property(&f)),
         );
 
     let path = object_path.get_name().to_owned();
@@ -211,6 +224,18 @@ pub fn get_blockdev_properties(
             consts::BLOCKDEV_NEW_SIZE_PROP => shared::blockdev_new_size_prop(dev)
         },
         consts::BLOCKDEV_INTERFACE_NAME_3_6 => {
+            consts::BLOCKDEV_DEVNODE_PROP => shared::blockdev_devnode_prop(dev),
+            consts::BLOCKDEV_HARDWARE_INFO_PROP => shared::blockdev_hardware_info_prop(dev),
+            consts::BLOCKDEV_USER_INFO_PROP => shared::blockdev_user_info_prop(dev),
+            consts::BLOCKDEV_INIT_TIME_PROP => shared::blockdev_init_time_prop(dev),
+            consts::BLOCKDEV_POOL_PROP => parent.clone(),
+            consts::BLOCKDEV_UUID_PROP => uuid_to_string!(dev_uuid),
+            consts::BLOCKDEV_TIER_PROP => shared::blockdev_tier_prop(tier),
+            consts::BLOCKDEV_PHYSICAL_PATH_PROP => shared::blockdev_physical_path_prop(dev),
+            consts::BLOCKDEV_TOTAL_SIZE_PROP => shared::blockdev_size_prop(dev),
+            consts::BLOCKDEV_NEW_SIZE_PROP => shared::blockdev_new_size_prop(dev)
+        },
+        consts::BLOCKDEV_INTERFACE_NAME_3_7 => {
             consts::BLOCKDEV_DEVNODE_PROP => shared::blockdev_devnode_prop(dev),
             consts::BLOCKDEV_HARDWARE_INFO_PROP => shared::blockdev_hardware_info_prop(dev),
             consts::BLOCKDEV_USER_INFO_PROP => shared::blockdev_user_info_prop(dev),

src/dbus_api/consts.rs

@@ -14,13 +14,15 @@ pub const MANAGER_INTERFACE_NAME_3_3: &str = "org.storage.stratis3.Manager.r3";
 pub const MANAGER_INTERFACE_NAME_3_4: &str = "org.storage.stratis3.Manager.r4";
 pub const MANAGER_INTERFACE_NAME_3_5: &str = "org.storage.stratis3.Manager.r5";
 pub const MANAGER_INTERFACE_NAME_3_6: &str = "org.storage.stratis3.Manager.r6";
+pub const MANAGER_INTERFACE_NAME_3_7: &str = "org.storage.stratis3.Manager.r7";
 pub const REPORT_INTERFACE_NAME_3_0: &str = "org.storage.stratis3.Report.r0";
 pub const REPORT_INTERFACE_NAME_3_1: &str = "org.storage.stratis3.Report.r1";
 pub const REPORT_INTERFACE_NAME_3_2: &str = "org.storage.stratis3.Report.r2";
 pub const REPORT_INTERFACE_NAME_3_3: &str = "org.storage.stratis3.Report.r3";
 pub const REPORT_INTERFACE_NAME_3_4: &str = "org.storage.stratis3.Report.r4";
 pub const REPORT_INTERFACE_NAME_3_5: &str = "org.storage.stratis3.Report.r5";
 pub const REPORT_INTERFACE_NAME_3_6: &str = "org.storage.stratis3.Report.r6";
+pub const REPORT_INTERFACE_NAME_3_7: &str = "org.storage.stratis3.Report.r7";
 
 pub const LOCKED_POOLS_PROP: &str = "LockedPools";
 pub const STOPPED_POOLS_PROP: &str = "StoppedPools";
@@ -32,6 +34,7 @@ pub const POOL_INTERFACE_NAME_3_3: &str = "org.storage.stratis3.pool.r3";
 pub const POOL_INTERFACE_NAME_3_4: &str = "org.storage.stratis3.pool.r4";
 pub const POOL_INTERFACE_NAME_3_5: &str = "org.storage.stratis3.pool.r5";
 pub const POOL_INTERFACE_NAME_3_6: &str = "org.storage.stratis3.pool.r6";
+pub const POOL_INTERFACE_NAME_3_7: &str = "org.storage.stratis3.pool.r7";
 pub const POOL_NAME_PROP: &str = "Name";
 pub const POOL_UUID_PROP: &str = "Uuid";
 pub const POOL_HAS_CACHE_PROP: &str = "HasCache";
@@ -53,6 +56,7 @@ pub const FILESYSTEM_INTERFACE_NAME_3_3: &str = "org.storage.stratis3.filesystem
 pub const FILESYSTEM_INTERFACE_NAME_3_4: &str = "org.storage.stratis3.filesystem.r4";
 pub const FILESYSTEM_INTERFACE_NAME_3_5: &str = "org.storage.stratis3.filesystem.r5";
 pub const FILESYSTEM_INTERFACE_NAME_3_6: &str = "org.storage.stratis3.filesystem.r6";
+pub const FILESYSTEM_INTERFACE_NAME_3_7: &str = "org.storage.stratis3.filesystem.r7";
 pub const FILESYSTEM_NAME_PROP: &str = "Name";
 pub const FILESYSTEM_UUID_PROP: &str = "Uuid";
 pub const FILESYSTEM_USED_PROP: &str = "Used";
@@ -69,6 +73,7 @@ pub const BLOCKDEV_INTERFACE_NAME_3_3: &str = "org.storage.stratis3.blockdev.r3"
 pub const BLOCKDEV_INTERFACE_NAME_3_4: &str = "org.storage.stratis3.blockdev.r4";
 pub const BLOCKDEV_INTERFACE_NAME_3_5: &str = "org.storage.stratis3.blockdev.r5";
 pub const BLOCKDEV_INTERFACE_NAME_3_6: &str = "org.storage.stratis3.blockdev.r6";
+pub const BLOCKDEV_INTERFACE_NAME_3_7: &str = "org.storage.stratis3.blockdev.r7";
 pub const BLOCKDEV_DEVNODE_PROP: &str = "Devnode";
 pub const BLOCKDEV_HARDWARE_INFO_PROP: &str = "HardwareInfo";
 pub const BLOCKDEV_USER_INFO_PROP: &str = "UserInfo";
@@ -90,6 +95,7 @@ pub fn standard_pool_interfaces() -> Vec<String> {
         POOL_INTERFACE_NAME_3_4,
         POOL_INTERFACE_NAME_3_5,
         POOL_INTERFACE_NAME_3_6,
+        POOL_INTERFACE_NAME_3_7,
     ]
     .iter()
     .map(|s| (*s).to_string())
@@ -106,6 +112,7 @@ pub fn standard_filesystem_interfaces() -> Vec<String> {
         FILESYSTEM_INTERFACE_NAME_3_4,
         FILESYSTEM_INTERFACE_NAME_3_5,
         FILESYSTEM_INTERFACE_NAME_3_6,
+        FILESYSTEM_INTERFACE_NAME_3_7,
     ]
     .iter()
     .map(|s| (*s).to_string())
@@ -122,6 +129,7 @@ pub fn standard_blockdev_interfaces() -> Vec<String> {
         BLOCKDEV_INTERFACE_NAME_3_4,
         BLOCKDEV_INTERFACE_NAME_3_5,
         BLOCKDEV_INTERFACE_NAME_3_6,
+        BLOCKDEV_INTERFACE_NAME_3_7,
     ]
     .iter()
     .map(|s| (*s).to_string())

src/dbus_api/filesystem/mod.rs

@@ -113,6 +113,18 @@ pub fn create_dbus_filesystem<'a>(
                 .add_p(filesystem_3_0::size_property(&f))
                 .add_p(filesystem_3_0::used_property(&f))
                 .add_p(filesystem_3_6::size_limit_property(&f)),
+        )
+        .add(
+            f.interface(consts::FILESYSTEM_INTERFACE_NAME_3_7, ())
+                .add_m(filesystem_3_0::rename_method(&f))
+                .add_p(filesystem_3_0::devnode_property(&f))
+                .add_p(filesystem_3_0::name_property(&f))
+                .add_p(filesystem_3_0::pool_property(&f))
+                .add_p(filesystem_3_0::uuid_property(&f))
+                .add_p(filesystem_3_0::created_property(&f))
+                .add_p(filesystem_3_0::size_property(&f))
+                .add_p(filesystem_3_0::used_property(&f))
+                .add_p(filesystem_3_6::size_limit_property(&f)),
         );
 
     let path = object_path.get_name().to_owned();
@@ -185,6 +197,16 @@ pub fn get_fs_properties(
             consts::FILESYSTEM_USED_PROP => shared::fs_used_prop(fs)
         },
         consts::FILESYSTEM_INTERFACE_NAME_3_6 => {
+            consts::FILESYSTEM_NAME_PROP => shared::fs_name_prop(fs_name),
+            consts::FILESYSTEM_UUID_PROP => uuid_to_string!(fs_uuid),
+            consts::FILESYSTEM_DEVNODE_PROP => shared::fs_devnode_prop(fs, pool_name, fs_name),
+            consts::FILESYSTEM_POOL_PROP => parent.clone(),
+            consts::FILESYSTEM_CREATED_PROP => shared::fs_created_prop(fs),
+            consts::FILESYSTEM_SIZE_PROP => shared::fs_size_prop(fs),
+            consts::FILESYSTEM_USED_PROP => shared::fs_used_prop(fs),
+            consts::FILESYSTEM_SIZE_LIMIT_PROP => shared::fs_size_limit_prop(fs)
+        },
+        consts::FILESYSTEM_INTERFACE_NAME_3_7 => {
             consts::FILESYSTEM_NAME_PROP => shared::fs_name_prop(fs_name),
             consts::FILESYSTEM_UUID_PROP => uuid_to_string!(fs_uuid),
             consts::FILESYSTEM_DEVNODE_PROP => shared::fs_devnode_prop(fs, pool_name, fs_name),

src/dbus_api/pool/mod.rs

@@ -241,6 +241,36 @@ pub fn create_dbus_pool<'a>(
                 .add_p(pool_3_1::fs_limit_property(&f))
                 .add_p(pool_3_1::enable_overprov_property(&f))
                 .add_p(pool_3_1::no_alloc_space_property(&f)),
+        )
+        .add(
+            f.interface(consts::POOL_INTERFACE_NAME_3_7, ())
+                .add_m(pool_3_6::create_filesystems_method(&f))
+                .add_m(pool_3_0::destroy_filesystems_method(&f))
+                .add_m(pool_3_0::snapshot_filesystem_method(&f))
+                .add_m(pool_3_0::add_blockdevs_method(&f))
+                .add_m(pool_3_0::bind_clevis_method(&f))
+                .add_m(pool_3_0::unbind_clevis_method(&f))
+                .add_m(pool_3_5::init_cache_method(&f))
+                .add_m(pool_3_0::add_cachedevs_method(&f))
+                .add_m(pool_3_0::bind_keyring_method(&f))
+                .add_m(pool_3_0::unbind_keyring_method(&f))
+                .add_m(pool_3_0::rebind_keyring_method(&f))
+                .add_m(pool_3_0::rebind_clevis_method(&f))
+                .add_m(pool_3_0::rename_method(&f))
+                .add_m(pool_3_3::grow_physical_device_method(&f))
+                .add_p(pool_3_0::name_property(&f))
+                .add_p(pool_3_0::uuid_property(&f))
+                .add_p(pool_3_0::encrypted_property(&f))
+                .add_p(pool_3_0::avail_actions_property(&f))
+                .add_p(pool_3_0::key_desc_property(&f))
+                .add_p(pool_3_0::clevis_info_property(&f))
+                .add_p(pool_3_0::has_cache_property(&f))
+                .add_p(pool_3_0::alloc_size_property(&f))
+                .add_p(pool_3_0::used_size_property(&f))
+                .add_p(pool_3_0::total_size_property(&f))
+                .add_p(pool_3_1::fs_limit_property(&f))
+                .add_p(pool_3_1::enable_overprov_property(&f))
+                .add_p(pool_3_1::no_alloc_space_property(&f)),
         );
 
     let path = object_path.get_name().to_owned();
@@ -357,6 +387,21 @@ pub fn get_pool_properties(
             consts::POOL_FS_LIMIT_PROP => shared::pool_fs_limit(pool),
             consts::POOL_OVERPROV_PROP => shared::pool_overprov_enabled(pool),
             consts::POOL_NO_ALLOCABLE_SPACE_PROP => shared::pool_no_alloc_space(pool)
+        },
+        consts::POOL_INTERFACE_NAME_3_7 => {
+            consts::POOL_NAME_PROP => shared::pool_name_prop(pool_name),
+            consts::POOL_UUID_PROP => uuid_to_string!(pool_uuid),
+            consts::POOL_ENCRYPTED_PROP => shared::pool_enc_prop(pool),
+            consts::POOL_AVAIL_ACTIONS_PROP => shared::pool_avail_actions_prop(pool),
+            consts::POOL_KEY_DESC_PROP => shared::pool_key_desc_prop(pool),
+            consts::POOL_CLEVIS_INFO_PROP => shared::pool_clevis_info_prop(pool),
+            consts::POOL_HAS_CACHE_PROP => shared::pool_has_cache_prop(pool),
+            consts::POOL_ALLOC_SIZE_PROP => shared::pool_allocated_size(pool),
+            consts::POOL_TOTAL_USED_PROP => shared::pool_used_size(pool),
+            consts::POOL_TOTAL_SIZE_PROP => shared::pool_total_size(pool),
+            consts::POOL_FS_LIMIT_PROP => shared::pool_fs_limit(pool),
+            consts::POOL_OVERPROV_PROP => shared::pool_overprov_enabled(pool),
+            consts::POOL_NO_ALLOCABLE_SPACE_PROP => shared::pool_no_alloc_space(pool)
         }
     }
 }