Skip to content

Commit

Permalink
helmfile sync: speedup (#3052)
Browse files Browse the repository at this point in the history
- change liveness and readyness probes to start querying more quickly to
  see if cassandra is up. Instead of 90 - 120 seconds, if cassandra is
up earlier that should manifest itself in the setup time of 'make
kube-integration-setup'
- change helmfile for wire-server to wait for databases-ephemeral to be
  up before launching pods: cassandra-migration needs to have a working
cassandra anyway - the crashloop-backoff strategy leads to a lot of
waiting in between restarts; so it should be faster to wait for
cassandra to be up before attempting schema migrations
  • Loading branch information
jschaul authored Feb 2, 2023
1 parent 443f741 commit d182942
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 0 deletions.
1 change: 1 addition & 0 deletions changelog.d/5-internal/helm-setup
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
CI integration setup time should be reduced: tweak the way cassandra-ephemeral is started
11 changes: 11 additions & 0 deletions charts/cassandra-ephemeral/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,14 @@ cassandra-ephemeral:
seed_size: 1
max_heap_size: 2048M
heap_new_size: 1024M

livenessProbe:
initialDelaySeconds: 10
periodSeconds: 10
successThreshold: 1
failureThreshold: 15
readinessProbe:
initialDelaySeconds: 10
periodSeconds: 10
successThreshold: 1
failureThreshold: 15
4 changes: 4 additions & 0 deletions hack/helmfile.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,8 @@ releases:
value: {{ .Values.federationDomain }}
- name: brig.config.optSettings.setFederationDomainConfigs[0].domain
value: {{ .Values.federationDomainFed2 }}
needs:
- '{{ .Values.namespace }}-databases-ephemeral'

- name: '{{ .Values.namespace }}-wire-server-2'
namespace: '{{ .Values.namespaceFed2 }}'
Expand All @@ -145,3 +147,5 @@ releases:
value: {{ .Values.federationDomainFed2 }}
- name: brig.config.optSettings.setFederationDomainConfigs[0].domain
value: {{ .Values.federationDomain }}
needs:
- '{{ .Values.namespace }}-databases-ephemeral-2'

0 comments on commit d182942

Please sign in to comment.