Skip to content

Commit

Permalink
enable concurrency config on ingestMetricsConsumers (#1117)
Browse files Browse the repository at this point in the history
  • Loading branch information
naser13 authored Jan 31, 2024
1 parent c85cd5b commit 8a0df4d
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 3 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: 21.0.0
version: 21.1.0
appVersion: 23.11.2
dependencies:
- name: memcached
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,10 @@ spec:
args:
- "run"
- "ingest-metrics-parallel-consumer"
{{- if .Values.sentry.ingestMetricsConsumerPerf.concurrency }}
- "--processes"
- "{{ .Values.sentry.ingestMetricsConsumerPerf.concurrency }}"
{{- end }}
- "--ingest-profile"
- "performance"
{{- if .Values.sentry.ingestMetricsConsumerPerf.maxBatchSize }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,10 @@ spec:
args:
- "run"
- "ingest-metrics-parallel-consumer"
{{- if .Values.sentry.ingestMetricsConsumerRh.concurrency }}
- "--processes"
- "{{ .Values.sentry.ingestMetricsConsumerRh.concurrency }}"
{{- end }}
- "--ingest-profile"
- "release-health"
{{- if .Values.sentry.ingestMetricsConsumerRh.maxBatchSize }}
Expand Down
4 changes: 2 additions & 2 deletions sentry/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ sentry:

ingestMetricsConsumerPerf:
replicas: 1
# concurrency: 4
concurrency: 1
env: []
resources: {}
affinity: {}
Expand All @@ -253,7 +253,7 @@ sentry:

ingestMetricsConsumerRh:
replicas: 1
# concurrency: 4
concurrency: 1
env: []
resources: {}
affinity: {}
Expand Down

0 comments on commit 8a0df4d

Please sign in to comment.