[Helm chart] clean unused values and setup nodeselector and affinity (#…

…78)

* remove service and autoscaling values

* remove autoscaling value in deployment

* implementing of nodeSelector and affinity in the helm chart

* removing unnecessary variable Release.Namespace

* removing unnecessary variable Release.Namespace

chart/templates/configmap.yaml

@@ -2,7 +2,6 @@ apiVersion: v1
 kind: ConfigMap
 metadata:
   name: {{ .Values.configMapName }}
-  namespace: {{ .Release.Namespace }}
 data:
   # downscale for non-work hours
   EXCLUDE_NAMESPACES: '{{- join "," .Values.excludedNamespaces }}'

chart/templates/deployment.yaml

@@ -4,11 +4,8 @@ metadata:
   labels:
     {{- include "py-kube-downscaler.labels" . | nindent 4 }}
   name: {{ include "py-kube-downscaler.name" . }}
-  namespace: {{ .Release.Namespace }}
 spec:
-  {{- if not .Values.autoscaling.enabled }}
   replicas: {{ .Values.replicaCount }}
-  {{- end }}
   selector:
     matchLabels:
       {{- include "py-kube-downscaler.selectorLabels" . | nindent 6 }}
@@ -36,6 +33,14 @@ spec:
           {{- toYaml .Values.securityContext | nindent 10 }}
       securityContext:
         {{- toYaml .Values.podSecurityContext | nindent 8 }}
+    {{- with .Values.nodeSelector }}
+      nodeSelector:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}
+    {{- with .Values.affinity }}
+      affinity:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}
       {{- with .Values.tolerations }}
       tolerations:
         {{- toYaml . | nindent 8 }}

chart/templates/rbac.yaml

@@ -2,7 +2,6 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: {{ include "py-kube-downscaler.serviceAccountName" . }}
-  namespace: {{ .Release.Namespace }}
 ---
 {{- if .Values.constrainedDownscaler }}
 {{- if not .Values.constrainedNamespaces }}

chart/values.yaml

@@ -42,10 +42,6 @@ securityContext:
     drop:
       - ALL
 
-service:
-  type: ClusterIP
-  port: 80
-
 resources:
   limits:
     cpu: 500m
@@ -54,12 +50,6 @@ resources:
     cpu: 200m
     memory: 300Mi
 
-autoscaling:
-  enabled: false
-  minReplicas: 1
-  maxReplicas: 100
-  targetCPUUtilizationPercentage: 80
-
 nodeSelector: {}
 
 tolerations: []