chore(bors): merge pull request #494

494: fix(upgrade-job): deserialize default when localpv chart is disabled r=niladrih a=niladrih

Local PV Hostpath will be installed as a chart alongside Mayastor, and not as a dependency of Mayastor in `openebs/openebs` v4.0.1. The deserialize fails when the chart is installed with helm v3.13+ because disabled dependency values are absent in the values set for those helm versions.

Co-authored-by: Niladri Halder <[email protected]>

k8s/upgrade/src/bin/upgrade-job/helm/chart.rs

@@ -1045,7 +1045,7 @@ impl PromtailConfigClient {
 
 /// This is used to deserialize the helm values of the localpv-provisioner helm chart.
 #[derive(Default, Deserialize)]
-#[serde(rename_all(deserialize = "camelCase"))]
+#[serde(default, rename_all(deserialize = "camelCase"))]
 struct LocalpvProvisioner {
     release: LocalpvProvisionerRelease,
     localpv: LocalpvProvisionerLocalpv,
@@ -1073,6 +1073,7 @@ impl LocalpvProvisioner {
 /// chart.
 #[derive(Default, Deserialize)]
 struct LocalpvProvisionerRelease {
+    #[serde(default)]
     version: String,
 }
 
@@ -1087,6 +1088,7 @@ impl LocalpvProvisionerRelease {
 /// This is used to deserialize the 'localpv' yaml object in the localpv-provisioner helm chart.
 #[derive(Default, Deserialize)]
 struct LocalpvProvisionerLocalpv {
+    #[serde(default)]
     image: GenericImage,
 }
 
@@ -1115,6 +1117,7 @@ impl GenericImage {
 /// This is used to deserialize the 'helperPod' yaml object in the localpv-provisioner helm chart.
 #[derive(Default, Deserialize)]
 struct LocalpvProvisionerHelperPod {
+    #[serde(default)]
     image: GenericImage,
 }