Skip to content

Commit c5f1ce4

Browse files
prakharmathur82prakharmathur82
committed
feat: add otelcollector as sidecar for firehose
1 parent acd8ba4 commit c5f1ce4

File tree

5 files changed

+43
-3
lines changed

5 files changed

+43
-3
lines changed

‎stable/firehose/Chart.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,5 @@ apiVersion: v2
22
name: firehose
33
description: A Helm chart for deploying Firehose on Kubernetes
44
type: application
5-
version: 0.1.3
6-
appVersion: 0.7.1
5+
version: 0.1.4
6+
appVersion: 0.8.1

‎stable/firehose/README.md

+9-1
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,15 @@ The following table lists the configurable parameters of Firehose chart and thei
9393
| telegraf.resources.limits.memory | string | `"64Mi"` | telegraf container memory limit |
9494
| telegraf.resources.requests.cpu | string | `"50m"` | telegraf container cpu requests |
9595
| telegraf.resources.requests.memory | string | `"64Mi"` | telegraf container memory requests |
96-
96+
| otelcollector.enabled | bool | `false` | flag for enabling otelcollector |
97+
| otelcollector.image.pullPolicy | string | `"IfNotPresent"` | the Kubernetes [imagePullPolicy](https://kubernetes.io/docs/concepts/containers/images/#updating-images) value for otelcollector container |
98+
| otelcollector.image.repository | string | `"otel/opentelemetry-collector-contrib"` | docker repository to download otelcollector image |
99+
| otelcollector.image.tag | string | `"0.108.0"` | otelcollector docker image tag |
100+
| otelcollector.resources.limits.cpu | string | `"50m"` | otelcollector container cpu limit |
101+
| otelcollector.resources.limits.memory | string | `"64Mi"` | otelcollector container memory limit |
102+
| otelcollector.resources.requests.cpu | string | `"50m"` | otelcollector container cpu requests |
103+
| otelcollector.resources.requests.memory | string | `"64Mi"` | otelcollector container memory requests |
104+
| otelcollector.config | object | `` | otelcollector configurations
97105
---
98106

99107
Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`.

‎stable/firehose/templates/configmap-otelcollector.yaml

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
{{- if index .Values.otelcollector.enabled }}
2+
apiVersion: v1
3+
kind: ConfigMap
4+
metadata:
5+
name: {{ include "firehose.fullname" . }}-otelcollector-config
6+
namespace: {{ .Release.Namespace }}
7+
labels:
8+
labels: {{- range $key, $value := .Values.labels }}
9+
{{ $key }}: {{ $value }}
10+
{{- end }}
11+
data:
12+
otel-collector-config.yaml: |+
13+
{{ toYaml .Values.otelcollector.config | indent 4 }}
14+
{{- end }}

‎stable/firehose/templates/deployment.yaml

+15
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,16 @@ spec:
5858
- mountPath: /etc/telegraf
5959
name: telegraf-conf
6060
{{- end }}
61+
{{- if index .Values.otelcollector.enabled }}
62+
- name: otelcollector
63+
image: "{{ .Values.otelcollector.image.repository }}:{{ .Values.otelcollector.image.tag }}"
64+
imagePullPolicy: {{ .Values.otelcollector.image.pullPolicy }}
65+
resources:
66+
{{- toYaml .Values.otelcollector.resources | nindent 12 }}
67+
volumeMounts:
68+
- mountPath: /etc/otelcollector
69+
name: otelcollector-conf
70+
{{- end }}
6171
{{- if index .Values "init-firehose" "enabled" }}
6272
initContainers:
6373
- args:
@@ -81,6 +91,11 @@ spec:
8191
name: {{ include "firehose.fullname" . }}-telegraf-config
8292
name: telegraf-conf
8393
{{- end }}
94+
{{- if index .Values.otelcollector.enabled }}
95+
- configMap:
96+
name: {{ include "firehose.fullname" . }}-otelcollector-config
97+
name: otelcollector-conf
98+
{{- end }}
8499
{{- if index .Values "init-firehose" "enabled" }}
85100
- emptyDir: {}
86101
name: workdir

‎stable/firehose/values.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -76,3 +76,6 @@ telegraf:
7676
requests:
7777
cpu: 50m
7878
memory: 64Mi
79+
80+
otelcollector:
81+
enabled: false

0 commit comments

Comments
 (0)