helm: Update readyset to 0.3.1

- Adds a new/old database.type key to values.yaml that needs to be
  either "mysql" or "psql". We will eventually remove this as we phase
  out older version support but until then it shall remain.
- Update values.yaml to add a image.tag key to readyset.adapter and
  readyset.server allowing users to specify the specific container
  version. The default for now is latest, but we will adjust that in the
  near-term to follow a release cadence once we've established it.
- Update chart version to 0.3.1

Change-Id: I79935c8fe097d4113d95f89aa68a2dbab8343a1e
Reviewed-on: https://gerrit.readyset.name/c/readyset/+/5015
Tested-by: Buildkite CI
Reviewed-by: David Kim <[email protected]>
Reviewed-by: Alana Marzoev <[email protected]>

Author: imeyer

helm/readyset/Chart.yaml

@@ -7,7 +7,7 @@ annotations:
 type: application
 
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.3.0
+version: 0.3.1
 
 # appVersion is not expected to follow Semantic Versioning (https://semver.org)
 appVersion: "latest"

helm/readyset/templates/readyset-adapter-deployment.yaml

@@ -78,7 +78,7 @@ spec:
                 curl http://127.0.0.1:8500/v1/health/node/$(hostname) \
                 2>/dev/null | grep -E '".+"'
         - name: readyset-adapter
-          image: {{ .Values.readyset.adapter.image.repository | default "public.ecr.aws/readyset" }}/readyset:latest
+          image: {{ .Values.readyset.adapter.image.repository | default "public.ecr.aws/readyset" }}/readyset:{{ .Values.readyset.adapter.image.tag | default "latest"}}
           imagePullPolicy: Always
           securityContext:
             runAsGroup: 1000
@@ -100,6 +100,8 @@ spec:
             # Identify this as a Helm deployment to TelemetryReporter
             - name: DEPLOYMENT_ENV
               value: "helm"
+            - name: DATABASE_TYPE
+              value: "{{ .Values.readyset.database.type | default "mysql" }}"
             - name: AUTHORITY_ADDRESS
               value: "readyset-consul-server:8500"
             - name: AUTHORITY

helm/readyset/templates/readyset-server-statefulset.yaml

@@ -87,7 +87,7 @@ spec:
                 curl http://127.0.0.1:8500/v1/health/node/$(hostname) \
                 2>/dev/null | grep -E '".+"'
         - name: readyset-server
-          image: public.ecr.aws/readyset/readyset-server:latest
+          image: {{ .Values.readyset.server.image.repository | default "public.ecr.aws/readyset" }}/readyset-server:{{ .Values.readyset.server.image.tag | default "latest"}}
           env:
             # ReadySet Deployment Name (Unique per Consul Cluster)
             - name: DEPLOYMENT

helm/readyset/values.yaml

@@ -4,12 +4,15 @@
 # Disclaimer: Removing any of the following items from the list
 # will likely break things in unspectacular ways.
 readyset:
+  database:
+    type: mysql # or psql; Necessary to work with older versions of ReadySet
   adapter:
     ingress:
       enabled: true
     httpPort: 6034
     image:
       repository: # "public.ecr.aws/readyset" # No trailing slash
+      tag: "latest"
     port: 3306 # Or 5432
     resources:
       requests:
@@ -24,6 +27,10 @@ readyset:
         storage: "50Gi"
         cpu: 500m
         memory: "1Gi"
+    image:
+      repository: # "public.ecr.aws/readyset" # No trailing slash
+      tag: "latest"
+
 
 kubernetes:
   storageClass: # Leave empty to use default provisioner