Otel collector integration by M4itee · Pull Request #519 · netdata/helmchart

M4itee · 2026-01-23T15:31:10Z

Adding ability to get otel metrics from the kubernetes

CLAassistant · 2026-01-23T15:31:20Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Mateusz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

charts/netdata/Chart.yaml

Copilot

Pull request overview

This pull request adds OpenTelemetry Collector integration to the Netdata Helm chart, enabling the collection of logs from Kubernetes pods and forwarding them to a netdata instance via OTLP protocol.

Changes:

Added netdataOpentelemetry deployment and service for receiving OTLP data
Integrated opentelemetry-collector as a subchart dependency for collecting Kubernetes logs
Fixed YAML indentation issues in ingress hosts and child tolerations configurations

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
charts/netdata/values.yaml	Added netdataOpentelemetry configuration section and otel-collector subchart settings; fixed indentation issues
charts/netdata/templates/netdata-otel/service.yaml	Created service to expose OTLP endpoint on port 4317
charts/netdata/templates/netdata-otel/secrets.yaml	Created secrets template for OTEL configuration
charts/netdata/templates/netdata-otel/persistentvolumeclaim.yaml	Created PVC for OTEL log storage
charts/netdata/templates/netdata-otel/deployment.yaml	Created deployment for netdata instance with OTEL plugin enabled
charts/netdata/templates/netdata-otel/configmap.yaml	Created configmap template for OTEL configuration
charts/netdata/templates/_helpers.tpl	Added helper templates for netdataOpentelemetry configs management
charts/netdata/Chart.yaml	Added opentelemetry-collector subchart dependency
charts/netdata/Chart.lock	Generated lock file for subchart dependency
charts/netdata/charts/opentelemetry-collector-0.115.0.tgz	Added OpenTelemetry Collector subchart package

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/values.yaml

ilyam8 · 2026-01-24T20:49:01Z

@cubic-dev-ai review this PR

Copilot

Pull request overview

Copilot reviewed 8 out of 10 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/values.yaml

charts/netdata/templates/netdata-otel/deployment.yaml

charts/netdata/values.yaml

charts/netdata/templates/netdata-otel/deployment.yaml

witalisoft

make sure that otel is also enabled when we are running checks

charts/netdata/templates/netdata-otel/deployment.yaml

charts/netdata/values.yaml

M4itee · 2026-01-30T14:30:55Z

templates/netdata-README.md.gotmpl

+
+The following table lists the configurable parameters of the netdata chart and their default values.
+
+{{ template "chart.valuesTableHtml" . }}


HTML because it renders better the multiline comments and values + adds some extra feats like copy, colors. Any markdown parser will render it no problem anyway.

M4itee · 2026-01-30T14:41:19Z

.github/workflows/checks.yml

      - name: Run chart-testing (install)
        run: ct install --target-branch ${{ github.event.repository.default_branch }}
+
+      - name: Run chart-testing (install with OpenTelemetry)


an extra for easier problem catching, in theory we can just use this one and delete the regular install. Good thing is that at the end of the ct install there is a cleanup so they are both independent.

M4itee · 2026-01-30T14:58:34Z

Last thing left for now is to make docs a bit more readable, after that it should be ready for review

Copilot

Pull request overview

Copilot reviewed 12 out of 14 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/templates/netdata-otel/deployment.yaml

charts/netdata/values.yaml

.github/workflows/checks.yml

install steps in cheks now wait for resources, archive is no longer in the branch - it is pulled during the tests and release.

Copilot

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/release.yml

ilyam8 requested a review from Copilot January 24, 2026 20:38

Copilot started reviewing on behalf of ilyam8 January 24, 2026 20:39 View session

ilyam8 reviewed Jan 24, 2026

View reviewed changes

charts/netdata/Chart.yaml Outdated Show resolved Hide resolved

Copilot AI reviewed Jan 24, 2026

View reviewed changes

charts/netdata/values.yaml Show resolved Hide resolved

charts/netdata/values.yaml Outdated Show resolved Hide resolved

charts/netdata/values.yaml Show resolved Hide resolved

ilyam8 requested a review from Copilot January 24, 2026 20:51

Copilot started reviewing on behalf of ilyam8 January 24, 2026 20:52 View session

Copilot AI reviewed Jan 24, 2026

View reviewed changes

witalisoft reviewed Jan 26, 2026

View reviewed changes

charts/netdata/templates/netdata-otel/deployment.yaml Outdated Show resolved Hide resolved

charts/netdata/values.yaml Show resolved Hide resolved

Mateusz added 2 commits January 30, 2026 14:54

adding otel integration

5d715ff

fixing naming, configuration, mounts for netdata-otel feat

b2922fa

M4itee force-pushed the otel-collector-integration branch from c268d60 to f065b9d Compare January 30, 2026 13:54

changes to docs, checks, otel values

ac92e27

M4itee force-pushed the otel-collector-integration branch from f065b9d to ac92e27 Compare January 30, 2026 13:59

fixing deployment template for otel

4b1dbb2

M4itee force-pushed the otel-collector-integration branch from 5c3757b to 4b1dbb2 Compare January 30, 2026 14:28

M4itee commented Jan 30, 2026

View reviewed changes

Mateusz added 2 commits January 30, 2026 15:49

Removing debug log level from otel, testing doc checks

45d2cb3

Updating documentation

0c55758

values table in non-html

d0bd6a1

M4itee force-pushed the otel-collector-integration branch 4 times, most recently from c4cc83f to 73258ee Compare January 30, 2026 17:13

removing long desc from some values

138f84e

M4itee force-pushed the otel-collector-integration branch from 73258ee to 138f84e Compare January 30, 2026 17:16

ilyam8 requested a review from Copilot January 30, 2026 18:00

Copilot started reviewing on behalf of ilyam8 January 30, 2026 18:00 View session

Copilot AI reviewed Jan 30, 2026

View reviewed changes

charts/netdata/templates/netdata-otel/deployment.yaml Show resolved Hide resolved

charts/netdata/values.yaml Outdated Show resolved Hide resolved

charts/netdata/values.yaml Outdated Show resolved Hide resolved

.github/workflows/checks.yml Show resolved Hide resolved

M4itee force-pushed the otel-collector-integration branch 3 times, most recently from a15734d to 3a77eb2 Compare February 2, 2026 12:04

changes to workflows - release and checks

595e55a

install steps in cheks now wait for resources, archive is no longer in the branch - it is pulled during the tests and release.

M4itee force-pushed the otel-collector-integration branch 3 times, most recently from abfbb32 to 8705505 Compare February 2, 2026 12:35

adding security context

436a9cc

M4itee force-pushed the otel-collector-integration branch from 8705505 to 436a9cc Compare February 2, 2026 12:35

M4itee marked this pull request as ready for review February 2, 2026 14:15

M4itee requested review from a team as code owners February 2, 2026 14:15

M4itee requested review from ilyam8 and witalisoft February 2, 2026 14:15

ilyam8 requested a review from Copilot February 2, 2026 15:21

Copilot started reviewing on behalf of ilyam8 February 2, 2026 15:21 View session

Copilot AI reviewed Feb 2, 2026

View reviewed changes

witalisoft previously approved these changes Feb 3, 2026

View reviewed changes

.github/workflows/release.yml Outdated Show resolved Hide resolved

removing helm install step from release (not needed)

8760d58

M4itee dismissed witalisoft’s stale review via 8760d58 February 4, 2026 12:19

M4itee requested review from kanelatechnical and witalisoft February 4, 2026 12:25

witalisoft approved these changes Feb 4, 2026

View reviewed changes


		The following table lists the configurable parameters of the netdata chart and their default values.

		{{ template "chart.valuesTableHtml" . }}

Conversation

M4itee commented Jan 23, 2026

Uh oh!

CLAassistant commented Jan 23, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ilyam8 commented Jan 24, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

witalisoft left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

M4itee Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

M4itee Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

M4itee commented Jan 30, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

M4itee Jan 30, 2026 •

edited

Loading