fix: improve syntax

SocialGouv · Jun 4, 2024 · 711e5c1 · 711e5c1
1 parent 07a9971
commit 711e5c1
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/plugins/contrib/patches/40-add-jobs-affinity-and-tolerations.js b/plugins/contrib/patches/40-add-jobs-affinity-and-tolerations.js
@@ -12,16 +12,14 @@ module.exports = (manifests, options) => {
 
     const parentSpec = kind === "CronJob" ? manifest.spec.jobTemplate : manifest
 
-    parentSpec.spec.template.spec = parentSpec.spec.template.spec || {}
+    parentSpec.spec.template.spec ||= {}
     const templateSpec = parentSpec.spec.template.spec
 
     // Add or merge affinity
-    templateSpec.affinity = templateSpec.affinity || {}
-    templateSpec.affinity.nodeAffinity =
-      templateSpec.affinity.nodeAffinity || {}
-    templateSpec.affinity.nodeAffinity.preferredDuringSchedulingIgnoredDuringExecution =
-      templateSpec.affinity.nodeAffinity
-        .preferredDuringSchedulingIgnoredDuringExecution || []
+    templateSpec.affinity ||= {}
+    templateSpec.affinity.nodeAffinity ||= {}
+    templateSpec.affinity.nodeAffinity.preferredDuringSchedulingIgnoredDuringExecution ||=
+      []
     const existingPreferences =
       templateSpec.affinity.nodeAffinity
         .preferredDuringSchedulingIgnoredDuringExecution
@@ -38,7 +36,7 @@ module.exports = (manifests, options) => {
     }
 
     // Add or merge tolerations
-    templateSpec.tolerations = templateSpec.tolerations || []
+    templateSpec.tolerations ||= []
     const newTolerations = tolerationsToAdd.filter(
       (tolerationToAdd) =>
         !templateSpec.tolerations.some(