Allows the BTRFS RAID levels to be either lowercase or uppercase when
deserializing them from string values, i.e. currently only the config
value of `btrfs.raid` in auto-installer answer files.

Signed-off-by: Daniel Kral <d.k...@proxmox.com>
---
When we're already at it, let's do the same for btrfs...

Here's the output from `cargo expand` again, first for deserialization:

```
fn visit_str<__E>(
    self,
    __value: &str,
) -> _serde::__private::Result<Self::Value, __E>
where
    __E: _serde::de::Error,
{
    match __value {
        "RAID0" | "raid0" => _serde::__private::Ok(__Field::__field0),
        "RAID1" | "raid1" => _serde::__private::Ok(__Field::__field1),
        "RAID10" | "raid10" => {
            _serde::__private::Ok(__Field::__field2)
        }
        _ => {
            _serde::__private::Err(
                _serde::de::Error::unknown_variant(__value, VARIANTS),
            )
        }
    }
}
```

and second for serialization:

```
impl _serde::Serialize for BtrfsRaidLevel {
fn serialize<__S>(
    &self,
    __serializer: __S,
) -> _serde::__private::Result<__S::Ok, __S::Error>
where
    __S: _serde::Serializer,
{
    match *self {
        BtrfsRaidLevel::Raid0 => {
            _serde::Serializer::serialize_unit_variant(
                __serializer,
                "BtrfsRaidLevel",
                0u32,
                "RAID0",
            )
        }
        BtrfsRaidLevel::Raid1 => {
            _serde::Serializer::serialize_unit_variant(
                __serializer,
                "BtrfsRaidLevel",
                1u32,
                "RAID1",
            )
        }
        BtrfsRaidLevel::Raid10 => {
            _serde::Serializer::serialize_unit_variant(
                __serializer,
                "BtrfsRaidLevel",
                2u32,
                "RAID10",
            )
        }
    }
}
}
```

On some error (`btrfs.raid = "rAID10"`), the message is:

```
proxmox-auto-install-assistant validate-answer answer.toml
Error: Error parsing answer file: TOML parse error at line 21, column 14
   |
21 | btrfs.raid = "rAID10"
   |              ^^^^^^^^
unknown variant `rAID10`, expected one of `raid0`, `raid1`, `raid10`
```

 proxmox-installer-common/src/options.rs | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/proxmox-installer-common/src/options.rs 
b/proxmox-installer-common/src/options.rs
index 434ce26..58fadf8 100644
--- a/proxmox-installer-common/src/options.rs
+++ b/proxmox-installer-common/src/options.rs
@@ -14,8 +14,11 @@ use crate::utils::{CidrAddress, Fqdn};
 #[derive(Copy, Clone, Debug, Deserialize, Serialize, Eq, PartialEq)]
 #[serde(rename_all(deserialize = "lowercase", serialize = "UPPERCASE"))]
 pub enum BtrfsRaidLevel {
+    #[serde(alias = "RAID0")]
     Raid0,
+    #[serde(alias = "RAID1")]
     Raid1,
+    #[serde(alias = "RAID10")]
     Raid10,
 }
 
-- 
2.39.5



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

Reply via email to