proxmox-mirrors/proxmox
1
0
Fork 1
mirror of https://git.proxmox.com/git/proxmox synced 2025-05-30 21:51:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

tape: fix 'eject-before-unload' api type

Browse Source 
by converting the bool into an option, otherwise having the options not
set at all will fail the unload while deserializing with
'eject-before-unload is not optional'

Also if we can automatically decide this in the future, we can now
detect if the option was explicitely set or not.

Fixes: 99f24b20 ("fix #4904: tape changer: add option to eject before unload")
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2023-12-13 11:11:12 +01:00 committed by Dietmar Maurer
parent 99f24b2079
commit 2cfa31a218
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
pbs-api-types/src/tape/changer.rs
View File

@ -39,18 +39,21 @@ Import/Export, i.e. any media in those slots are considered to be
.format(&ApiStringFormat::PropertyString(&SLOT_ARRAY_SCHEMA)) .format(&ApiStringFormat::PropertyString(&SLOT_ARRAY_SCHEMA))
.schema(); .schema();
fn is_false(b: &bool) -> bool { #[api(
!b properties: {
} "eject-before-unload": {
optional: true,
#[api] default: false,
},
},
)]
#[derive(Serialize, Deserialize)] #[derive(Serialize, Deserialize)]
#[serde(rename_all = "kebab-case")] #[serde(rename_all = "kebab-case")]
/// Options for Changers /// Options for Changers
pub struct ChangerOptions { pub struct ChangerOptions {
#[serde(default, skip_serializing_if = "is_false")] #[serde(skip_serializing_if = "Option::is_none")]
/// if set to true, tapes are ejected manually before unloading /// if set to true, tapes are ejected manually before unloading
pub eject_before_unload: bool, pub eject_before_unload: Option<bool>,
} }
pub const CHANGER_OPTIONS_STRING_SCHEMA: Schema = StringSchema::new("Changer options") pub const CHANGER_OPTIONS_STRING_SCHEMA: Schema = StringSchema::new("Changer options")