+serde_plain::derive_deserialize_from_fromstr!(FirewallPacketRate, "valid packet rate"); +serde_plain::derive_serialize_from_display!(FirewallPacketRate);we have similar macros in proxmox-serde as well [1]. Would potentially save an external dependency? applies to all instances in this patch series ofc.
I think we should remove those macros, and use serde_plain instead.
