Browse Source

Updating to use preferred only for logging components and removing infra pod concept

Eric Wolinetz 7 years ago
parent
commit
363e2a0fe5

+ 3 - 17
roles/openshift_logging_elasticsearch/templates/2.x/es.j2

@@ -29,30 +29,16 @@ spec:
     spec:
       affinity:
         podAntiAffinity:
-          requiredDuringSchedulingIgnoredDuringExecution:
-          - labelSelector:
-              matchExpressions:
-              - key: logging-infra
-                operator: In
-                values:
-                - elasticsearch
-            topologyKey: kubernetes.io/hostname
-{% if openshift_infra_anti_affinity_labels is defined and openshift_infra_anti_affinity_labels is iterable and openshift_infra_anti_affinity_labels | length > 0 %}
           preferredDuringSchedulingIgnoredDuringExecution:
           - weight: 100
             podAffinityTerm:
               labelSelector:
                 matchExpressions:
-{% for aa_key, aa_values in openshift_infra_anti_affinity_labels.items() %}
-                - key: "{{ aa_key }}"
+                - key: logging-infra
                   operator: In
                   values:
-{% for aa_value in aa_values %}
-                  - "{{ aa_value }}"
-{% endfor %}
-{% endfor %}
-              topologyKey: kubernetes.io/hostname
-{% endif %}
+                  - elasticsearch
+            topologyKey: kubernetes.io/hostname
       terminationGracePeriod: 600
       serviceAccountName: aggregated-logging-elasticsearch
       securityContext:

+ 3 - 17
roles/openshift_logging_elasticsearch/templates/5.x/es.j2

@@ -29,30 +29,16 @@ spec:
     spec:
       affinity:
         podAntiAffinity:
-          requiredDuringSchedulingIgnoredDuringExecution:
-          - labelSelector:
-              matchExpressions:
-              - key: logging-infra
-                operator: In
-                values:
-                - elasticsearch
-            topologyKey: kubernetes.io/hostname
-{% if openshift_infra_anti_affinity_labels is defined and openshift_infra_anti_affinity_labels is iterable and openshift_infra_anti_affinity_labels | length > 0 %}
           preferredDuringSchedulingIgnoredDuringExecution:
           - weight: 100
             podAffinityTerm:
               labelSelector:
                 matchExpressions:
-{% for aa_key, aa_values in openshift_infra_anti_affinity_labels.items() %}
-                - key: "{{ aa_key }}"
+                - key: logging-infra
                   operator: In
                   values:
-{% for aa_value in aa_values %}
-                  - "{{ aa_value }}"
-{% endfor %}
-{% endfor %}
-              topologyKey: kubernetes.io/hostname
-{% endif %}
+                  - elasticsearch
+            topologyKey: kubernetes.io/hostname
       terminationGracePeriod: 600
       serviceAccountName: aggregated-logging-elasticsearch
       securityContext:

+ 9 - 23
roles/openshift_logging_kibana/templates/2.x/kibana.j2

@@ -28,30 +28,16 @@ spec:
     spec:
       affinity:
         podAntiAffinity:
-          requiredDuringSchedulingIgnoredDuringExecution:
-          - labelSelector:
-              matchExpressions:
-              - key: logging-infra
-                operator: In
-                values:
-                - kibana
+          preferredDuringSchedulingIgnoredDuringExecution:
+          - weight: 100
+            podAffinityTerm:
+              labelSelector:
+                matchExpressions:
+                - key: logging-infra
+                  operator: In
+                  values:
+                  - kibana
             topologyKey: kubernetes.io/hostname
-{% if openshift_infra_anti_affinity_labels is defined and openshift_infra_anti_affinity_labels is iterable and openshift_infra_anti_affinity_labels | length > 0 %}
-            preferredDuringSchedulingIgnoredDuringExecution:
-            - weight: 100
-              podAffinityTerm:
-                labelSelector:
-                  matchExpressions:
-{% for aa_key, aa_values in openshift_infra_anti_affinity_labels.items() %}
-                  - key: "{{ aa_key }}"
-                    operator: In
-                    values:
-{% for aa_value in aa_values %}
-                    - "{{ aa_value }}"
-{% endfor %}
-{% endfor %}
-                topologyKey: kubernetes.io/hostname
-{% endif %}
       serviceAccountName: aggregated-logging-kibana
 {% if kibana_node_selector is iterable and kibana_node_selector | length > 0 %}
       nodeSelector:

+ 5 - 19
roles/openshift_logging_kibana/templates/5.x/kibana.j2

@@ -28,30 +28,16 @@ spec:
     spec:
       affinity:
         podAntiAffinity:
-          requiredDuringSchedulingIgnoredDuringExecution:
-          - labelSelector:
+        preferredDuringSchedulingIgnoredDuringExecution:
+        - weight: 100
+          podAffinityTerm:
+            labelSelector:
               matchExpressions:
               - key: logging-infra
                 operator: In
                 values:
                 - kibana
-            topologyKey: kubernetes.io/hostname
-{% if openshift_infra_anti_affinity_labels is defined and openshift_infra_anti_affinity_labels is iterable and openshift_infra_anti_affinity_labels | length > 0 %}
-            preferredDuringSchedulingIgnoredDuringExecution:
-            - weight: 100
-              podAffinityTerm:
-                labelSelector:
-                  matchExpressions:
-{% for aa_key, aa_values in openshift_infra_anti_affinity_labels.items() %}
-                  - key: "{{ aa_key }}"
-                    operator: In
-                    values:
-{% for aa_value in aa_values %}
-                    - "{{ aa_value }}"
-{% endfor %}
-{% endfor %}
-                topologyKey: kubernetes.io/hostname
-{% endif %}
+          topologyKey: kubernetes.io/hostname
       serviceAccountName: aggregated-logging-kibana
 {% if kibana_node_selector is iterable and kibana_node_selector | length > 0 %}
       nodeSelector: