[jspolicy] fix var name in pod volume mounts (#898)

* [jspolicy] fix var name in pod volume mounts * bump version
truefoundry · Dec 24, 2024 · 1e40f36 · 1e40f36
1 parent ff31b53
commit 1e40f36
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 22 deletions.
diff --git a/charts/tfy-jspolicy-configs/Chart.yaml b/charts/tfy-jspolicy-configs/Chart.yaml
@@ -2,4 +2,4 @@ apiVersion: v2
 name: tfy-jspolicy-config
 description: A Helm chart for jspolicy configuration
 type: application
-version: 0.2.1
+version: 0.2.2
diff --git a/charts/tfy-jspolicy-configs/templates/pod-volume-mounts.yaml b/charts/tfy-jspolicy-configs/templates/pod-volume-mounts.yaml
@@ -37,45 +37,45 @@ spec:
     print("Adding volume mounts to the pod " + request.object.metadata.name + " in namespace " + request.namespace);
     // add the volumes to the pod
     const podVolumes = request.object.spec.volumes || [];
-    mountDetails.forEach(mountDetails => {
-      if (mountDetails.type === "secret") {
+    mountDetails.forEach(mountDetail => {
+      if (mountDetail.type === "secret") {
         // skip if the volume already exists, volume name is same as secret name
-        if (podVolumes.find(volume => volume.name === mountDetails.secretName)) {
-          print("Volume " + mountDetails.secretName + " already exists in the pod " + request.object.metadata.name + " in namespace " + request.namespace + ". Skipping...");
+        if (podVolumes.find(volume => volume.name === mountDetail.secretName)) {
+          print("Volume " + mountDetail.secretName + " already exists in the pod " + request.object.metadata.name + " in namespace " + request.namespace + ". Skipping...");
           return;
         }
-        print("Adding volume " + mountDetails.secretName + " to the pod " + request.object.metadata.name + " in namespace " + request.namespace);
+        print("Adding volume " + mountDetail.secretName + " to the pod " + request.object.metadata.name + " in namespace " + request.namespace);
         request.object.spec.volumes = request.object.spec.volumes.concat({
-          name: mountDetails.secretName,
+          name: mountDetail.secretName,
         secret: {
-          secretName: mountDetails.secretName
+          secretName: mountDetail.secretName
           }
         });
       }
-      if (mountDetails.type === "configMap") {
+      if (mountDetail.type === "configMap") {
         // skip if the volume already exists, volume name is same as secret name
-        if (podVolumes.find(volume => volume.name === mountDetails.configMapName)) {
-          print("Volume " + mountDetails.configMapName + " already exists in the pod " + request.object.metadata.name + " in namespace " + request.namespace + ". Skipping...");
+        if (podVolumes.find(volume => volume.name === mountDetail.configMapName)) {
+          print("Volume " + mountDetail.configMapName + " already exists in the pod " + request.object.metadata.name + " in namespace " + request.namespace + ". Skipping...");
           return;
         }
-        print("Adding volume " + mountDetails.configMapName + " to the pod " + request.object.metadata.name + " in namespace " + request.namespace);
+        print("Adding volume " + mountDetail.configMapName + " to the pod " + request.object.metadata.name + " in namespace " + request.namespace);
         request.object.spec.volumes = request.object.spec.volumes.concat({
-          name: mountDetails.configMapName,
+          name: mountDetail.configMapName,
           configMap: {
-            name: mountDetails.configMapName
+            name: mountDetail.configMapName
           }
         });
       }
       request.object.spec.containers.forEach(container => {
         let volumeName = "";
-        if (mountDetails.type === "secret") {
-          volumeName = mountDetails.secretName;
+        if (mountDetail.type === "secret") {
+          volumeName = mountDetail.secretName;
         }
-        if (mountDetails.type === "configMap") {
-          volumeName = mountDetails.configMapName;
+        if (mountDetail.type === "configMap") {
+          volumeName = mountDetail.configMapName;
         }
         if (volumeName === "") {
-          print("Invalid volume mount details: " + JSON.stringify(mountDetails));
+          print("Invalid volume mount details: " + JSON.stringify(mountDetail));
           exit();
         }
         // skip if the volume mount already exists
@@ -84,11 +84,11 @@ spec:
           return;
         }
         print("Adding volume mount " + volumeName + " to the container " + container.name + " in the pod " + request.object.metadata.name + " in namespace " + request.namespace);
-        print("Mount details: " + JSON.stringify(mountDetails));
+        print("Mount details: " + JSON.stringify(mountDetail));
         container.volumeMounts = container.volumeMounts.concat({
           name: volumeName,
-          mountPath: mountDetails.mountPath,
-          subPath: mountDetails.subPath || "",
+          mountPath: mountDetail.mountPath,
+          subPath: mountDetail.subPath || "",
           readOnly: true
           });
       });