pfab 0.53.0 → 0.54.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 06aa820e3141930ca45a670075b6824a23010f5f3568aa3c81edd1825ea452c2
-  data.tar.gz: 9d43a1c6d16ac3e44f057817fa50f0143d7e3ce6d9b9ca0446735ae060144dfe
+  metadata.gz: f6db80ed30cdab582adbf60a44b1cb80ffb32420898d07117b5e0fe81f0012ac
+  data.tar.gz: 818d93a741088cee1a7988273e6eb3389c013fee43227193d086f9cdaa8c01ba
 SHA512:
-  metadata.gz: 9e42f6a07719549799f99885888d33218dd69b0211e07ea7d927bf22b4d6d39984a4d246aa7f1c99afa8ccdd273eb2d96292363cb167b0a7c1c09e7ecff73464
-  data.tar.gz: 655b9ecb7007a4c19e27591a6edbe2acefcbce8a374ade1b1fb2e97985089503ae875d156cf269d969984f02e1dc148ca1e86a7702c444aab8e5b33fd65bc288
+  metadata.gz: f53b2f212245784bd4d518c7058aa7a2bdad7c2bba7b4a1c956f86356e037ff429ec704c1f7210969ac05e2bfbc05e2273e7961cee9028941e0361d85aaaa618
+  data.tar.gz: 2d1e0b3cd52341158feea24b3f184b5ccd53c1dd1951b37b4cdcbed9d54d5e023c7bd772e27a7f86828f56ad86ccf1187c7ed7596a79acc5efc6457d525199cd

data/lib/pfab/templates/web.rb

@@ -187,10 +187,7 @@ module Pfab
 
 
       def anti_affinity
-        p = host_anti_affinity
-        z = zone_anti_affinity
-        m =  merge_anti_affinity(p, z)
-        return m
+        return host_anti_affinity
       end
 
 
@@ -198,63 +195,57 @@ module Pfab
         anti_affinity_builder(ANTI_AFFINITY_MODE, ANTI_AFFINITY_PREFERRED_MODE_WEIGHT, "kubernetes.io/hostname")
       end
 
-      def zone_anti_affinity
-        anti_affinity_builder(ZONE_ANTI_AFFINITY_MODE, ZONE_ANTI_AFFINITY_PREFERRED_MODE_WEIGHT, "topology.kubernetes.io/zone")
-      end
 
-      def merge_anti_affinity(pod_anti_affinity, zone_anti_affinity)
-        merged = {}
+      def topology_spread_constraints
+        waiveTopologySpreadConstraints = get("waiveTopologySpreadConstraints") || false
 
-        if pod_anti_affinity.empty? && zone_anti_affinity.empty?
-          merged = {}
-        elsif pod_anti_affinity.empty?
-          merged = zone_anti_affinity
-        elsif zone_anti_affinity.empty?
-          merged = pod_anti_affinity
-        else
-          merged[:podAntiAffinity] = {}
-
-          required_key = :requiredDuringSchedulingIgnoredDuringExecution
-          preferred_key = :preferredDuringSchedulingIgnoredDuringExecution
-
-          [required_key, preferred_key].each do |key|
-            if pod_anti_affinity.dig(:podAntiAffinity, key) && zone_anti_affinity.dig(:podAntiAffinity, key)
-              merged[:podAntiAffinity][key] = pod_anti_affinity[:podAntiAffinity][key] + zone_anti_affinity[:podAntiAffinity][key]
-            elsif pod_anti_affinity.dig(:podAntiAffinity, key)
-              merged[:podAntiAffinity][key] = pod_anti_affinity[:podAntiAffinity][key]
-            elsif zone_anti_affinity.dig(:podAntiAffinity, key)
-              merged[:podAntiAffinity][key] = zone_anti_affinity[:podAntiAffinity][key]
-            end
-          end
-        end
+        schedulingRule = waiveTopologySpreadConstraints ? "ScheduleAnyway" : "DoNotSchedule"
+
+        zone_constraint = {
+          maxSkew: 1,
+          topologyKey: "topology.kubernetes.io/zone",
+          whenUnsatisfiable: schedulingRule,
+          labelSelector: labelSelector
+        }
+        host_constraint = {
+          maxSkew: 1,
+          topologyKey: "kubernetes.io/hostname",
+          whenUnsatisfiable: schedulingRule,
+          labelSelector: labelSelector
 
-        merged
+        }
+        [zone_constraint, host_constraint]
       end
 
 
+
+      def labelSelector
+        {
+          matchExpressions: [
+          {
+            key: "deployed-name",
+            operator: "In",
+            values: [
+              @data['deployed_name']
+            ]
+          },
+          {
+            key: "deployment-unique-id",
+            operator: "In",
+            values: [
+              StyledYAML.double_quoted(deploy_unique_id)
+            ]
+          }
+        ]
+        }
+      end
+
       def anti_affinity_builder(key, weight_key, topology_key)
         antiAffinityMode = get(key) || "disabled"
         if antiAffinityMode
           affinitySelector = {
             topologyKey: topology_key,
-            labelSelector: {
-              matchExpressions: [
-                {
-                  key: "deployed-name",
-                  operator: "In",
-                  values: [
-                    @data['deployed_name']
-                  ]
-                },
-                {
-                  key: "deployment-unique-id",
-                  operator: "In",
-                  values: [
-                    StyledYAML.double_quoted(deploy_unique_id)
-                  ]
-                }
-              ]
-            },
+            labelSelector: labelSelector,
           }
 
           return case antiAffinityMode
@@ -381,7 +372,7 @@ module Pfab
                     volumeMounts: volume_mounts
                   }.compact
                 ],
-                affinity:  anti_affinity,
+                topologySpreadConstraints: topology_spread_constraints,
                 volumes: volumes
               }.compact,
             },

data/lib/pfab/version.rb

@@ -1,7 +1,7 @@
 module Pfab
   module Version
     MAJOR = 0
-    MINOR = 53
+    MINOR = 54
     PATCH = 0
     BUILD = nil
 

data/pfab.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: pfab 0.53.0 ruby lib
+# stub: pfab 0.54.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "pfab".freeze
-  s.version = "0.53.0"
+  s.version = "0.54.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["Jeff Dwyer".freeze]
-  s.date = "2024-05-14"
+  s.date = "2024-05-15"
   s.description = "k8s helper".freeze
   s.email = "jdwyer@prefab.cloud".freeze
   s.executables = ["pfab".freeze]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pfab
 version: !ruby/object:Gem::Version
-  version: 0.53.0
+  version: 0.54.0
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-14 00:00:00.000000000 Z
+date: 2024-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: commander