client: task polling: rework code to be slightly more readable

the match-arms let one follow the different branches easier than the
relatively crowded if-condition it replaces.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2023-09-27 18:14:57 +02:00
parent af7b07479b
commit c5e54a5dca

View File

@ -97,12 +97,12 @@ pub async fn display_task_log(
} }
let status_path = format!("api2/json/nodes/localhost/tasks/{upid_encoded}/status"); let status_path = format!("api2/json/nodes/localhost/tasks/{upid_encoded}/status");
let status_result = client.get(&status_path, None).await?; let task_result = &client.get(&status_path, None).await?["data"];
if status_result["data"]["status"].as_str() == Some("stopped") { if task_result["status"].as_str() == Some("stopped") {
if let Some(status) = status_result["data"]["exitstatus"].as_str() { match task_result["exitstatus"].as_str() {
if status != "OK" && !status.starts_with("WARNINGS") { None => bail!("task stopped with unknown status"),
bail!("task failed"); Some(status) if status == "OK" || status.starts_with("WARNINGS") => (),
} Some(status) => bail!("task failed (status {status})"),
} }
} }