[pbs-devel] [PATCH proxmox-backup] fix #5946: api: disks: directory: fix unknown task status when partition creation fails

[pbs-devel] [PATCH proxmox-backup] fix #5946: api: disks: directory: fix unknown task status when partition creation fails

[Filip Schauer]

This shows a more helpful error status than "unknown", when the
directory creation task fails during creation of the partition on disk.
The full error message can still be viewed in the task log.

Signed-off-by: Filip Schauer <f.schauer@proxmox.com>
---
Is there a cleaner way to handle this?

 src/api2/node/disks/directory.rs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/api2/node/disks/directory.rs b/src/api2/node/disks/directory.rs
index 62f46343..cd60f192 100644
--- a/src/api2/node/disks/directory.rs
+++ b/src/api2/node/disks/directory.rs
@@ -223,7 +223,13 @@ pub fn create_datastore_disk(
 
             let disk = manager.disk_by_name(&disk)?;
 
-            let partition = create_single_linux_partition(&disk)?;
+            let partition = match create_single_linux_partition(&disk) {
+                Ok(partition) => partition,
+                Err(err) => {
+                    info!("{err}");
+                    bail!("failed to format disk as Linux partition");
+                }
+            };
             create_file_system(&partition, filesystem)?;
 
             let uuid = get_fs_uuid(&partition)?;
-- 
2.39.5


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel