[synthetic-monitoring-agent] fix deployment not starting on update/auto-scaling

charts/synthetic-monitoring-agent/Chart.yaml

@@ -15,4 +15,4 @@ name: synthetic-monitoring-agent
 sources:
   - https://github.com/grafana/synthetic-monitoring-agent
 type: application
-version: 0.2.0
+version: 0.3.0

charts/synthetic-monitoring-agent/README.md

@@ -32,11 +32,6 @@ Kubernetes: `^1.16.0-0`
 | agent.debug | bool | `false` | Enable / disable debug logging on the agent. |
 | agent.enableDisconnect | bool | `false` | Enable / disable the HTTP /disconnect endpoint |
 | agent.verbose | bool | `false` | Enable / disable verbose logging on the agent. |
-| autoscaling.enabled | bool | `false` | Enable autoscaling |
-| autoscaling.maxReplicas | int | `3` | Maximum autoscaling replicas |
-| autoscaling.minReplicas | int | `1` | Minimum autoscaling replicas |
-| autoscaling.targetCPUUtilizationPercentage | int | `60` | Target CPU utilisation percentage |
-| autoscaling.targetMemoryUtilizationPercentage | string | `nil` | Target memory utilisation percentage |
 | extraObjects | list | `[]` | Add dynamic manifests via values: |
 | fullnameOverride | string | `""` | Override the fullname of the chart. |
 | hostAliases | list | `[]` | hostAliases to add |
@@ -52,7 +47,6 @@ Kubernetes: `^1.16.0-0`
 | podLabels | object | `{}` | Labels to add to each pod. |
 | podSecurityContext | object | `{"fsGroup":65534}` | Security context on the Pod level. |
 | readinessProbe | object | `{"httpGet":{"path":"/ready","port":"http"}}` | Readiness probe for the agent |
-| replicaCount | int | `1` | Number of replicas to use; ignored if `autoscaling.enabled` is set to `true`. |
 | resources | object | `{}` | Default resources to apply. |
 | securityContext | object | `{"capabilities":{"drop":["ALL"]},"readOnlyRootFilesystem":true,"runAsNonRoot":true,"runAsUser":65534}` | Security context for the container level. |
 | service.annotations | object | `{}` | Annotations for the service |

charts/synthetic-monitoring-agent/templates/deployment.yaml → charts/synthetic-monitoring-agent/templates/statefulset.yaml

@@ -1,14 +1,13 @@
 apiVersion: apps/v1
-kind: Deployment
+kind: StatefulSet
 metadata:
   name: {{ include "synthetic-monitoring-agent.fullname" . }}
   labels:
     {{- include "synthetic-monitoring-agent.labels" . | nindent 4 }}
 spec:
   minReadySeconds: 10
-  {{- if not .Values.autoscaling.enabled }}
-  replicas: {{ .Values.replicaCount }}
-  {{- end }}
+  replicas: 1
+  serviceName: {{ include "synthetic-monitoring-agent.fullname" . }}-headless
   selector:
     matchLabels:
       {{- include "synthetic-monitoring-agent.selectorLabels" . | nindent 6 }}

charts/synthetic-monitoring-agent/values.yaml

@@ -15,9 +15,6 @@ agent:
   # -- Enable / disable verbose logging on the agent.
   verbose: false
 
-# -- Number of replicas to use; ignored if `autoscaling.enabled` is set to `true`.
-replicaCount: 1
-
 image:
   # -- Base registry to pull the container image from.
   registry: docker.io
@@ -72,18 +69,6 @@ resources:
   #   cpu: 200m
   #   memory: 512Mi
 
-autoscaling:
-  # -- Enable autoscaling
-  enabled: false
-  # -- Minimum autoscaling replicas
-  minReplicas: 1
-  # -- Maximum autoscaling replicas
-  maxReplicas: 3
-  # -- Target CPU utilisation percentage
-  targetCPUUtilizationPercentage: 60
-  # -- Target memory utilisation percentage
-  targetMemoryUtilizationPercentage:
-
 # -- Liveness probe for the agent
 livenessProbe:
   httpGet: