[PATCH proxmox-backup v2 02/10] api: config: rearrange independent code block for datastore creation

[Christian Ebner <c.ebner@proxmox.com>]

In preparation for moving most of the datastore creation logic from
the worker task helper to the datastore implementation itself.

The unmount guard is only crated for removable datastores, so it can
also be placed before the backend type check and s3 client
instantiation.

No functional changes.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
---
 src/api2/config/datastore.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/api2/config/datastore.rs b/src/api2/config/datastore.rs
index 16e85a636..7c3531a4b 100644
--- a/src/api2/config/datastore.rs
+++ b/src/api2/config/datastore.rs
@@ -126,6 +126,13 @@ pub(crate) fn do_create_datastore(
             .parse_property_string(datastore.tuning.as_deref().unwrap_or(""))?,
     )?;
 
+    let unmount_guard = if datastore.backing_device.is_some() {
+        do_mount_device(datastore.clone())?;
+        UnmountGuard::new(Some(path.clone()))
+    } else {
+        UnmountGuard::new(None)
+    };
+
     let (backend_type, backend_s3_client) =
         match DataStore::s3_client_and_backend_from_datastore_config(&datastore)? {
             (backend_type, Some(s3_client)) => {
@@ -155,13 +162,6 @@ pub(crate) fn do_create_datastore(
             (backend_type, None) => (backend_type, None),
         };
 
-    let unmount_guard = if datastore.backing_device.is_some() {
-        do_mount_device(datastore.clone())?;
-        UnmountGuard::new(Some(path.clone()))
-    } else {
-        UnmountGuard::new(None)
-    };
-
     let chunk_store = if reuse_datastore && backend_type == DatastoreBackendType::Filesystem {
         ChunkStore::verify_chunkstore(&path).and_then(|_| {
             // Must be the only instance accessing and locking the chunk store,
-- 
2.47.3