Skip to content

Commit

Permalink
* Fix filesystem PVC creation with filestore backend not equal to "fi…
Browse files Browse the repository at this point in the history
…lesystem" (#242)

* Fix double replacer: declaration in values.yaml
  • Loading branch information
Antiarchitect authored Nov 16, 2020
1 parent b3b662b commit 95675e3
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion sentry/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: sentry
description: A Helm chart for Kubernetes
type: application
version: 7.3.0
version: 7.3.1
appVersion: 20.10.1
dependencies:
- name: redis
Expand Down
2 changes: 2 additions & 0 deletions sentry/templates/pvc.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{{- if eq .Values.filestore.backend "filesystem" -}}
{{- if and .Values.filestore.filesystem.persistence.enabled (not .Values.filestore.filesystem.persistence.existingClaim) -}}
kind: PersistentVolumeClaim
apiVersion: v1
Expand All @@ -22,3 +23,4 @@ spec:
{{- end }}
{{- end }}
{{- end -}}
{{- end -}}
10 changes: 4 additions & 6 deletions sentry/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -148,9 +148,6 @@ snuba:
maxReplicas: 5
targetCPUUtilizationPercentage: 50

replacer:
replicas: 1

consumer:
replicas: 1
env: []
Expand All @@ -169,7 +166,7 @@ snuba:
# tolerations: []
# podLabels: []

sessionsConsumer:
replacer:
replicas: 1
env: []
resources: {}
Expand All @@ -178,7 +175,7 @@ snuba:
# tolerations: []
# podLabels: []

transactionsConsumer:
sessionsConsumer:
replicas: 1
env: []
resources: {}
Expand All @@ -187,7 +184,8 @@ snuba:
# tolerations: []
# podLabels: []

replacer:
transactionsConsumer:
replicas: 1
env: []
resources: {}
affinity: {}
Expand Down

0 comments on commit 95675e3

Please sign in to comment.