Skip to content

feat: add trafficDistribution support to opentelemetry-kube-stack - #2081

Closed
kema-dev wants to merge 5 commits into
open-telemetry:mainfrom
kema-dev:feat-add-traffic-distribution
Closed

feat: add trafficDistribution support to opentelemetry-kube-stack#2081
kema-dev wants to merge 5 commits into
open-telemetry:mainfrom
kema-dev:feat-add-traffic-distribution

Conversation

@kema-dev

Copy link
Copy Markdown

Description

Possible improvements

  • We could add PreferSameNode as value for the daemon (default) collector, but that would require a fairly recent Kubernetes version

@kema-dev
kema-dev requested review from a team and jaronoff97 as code owners February 16, 2026 16:48
@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Mar 3, 2026
@kema-dev

kema-dev commented Mar 3, 2026

Copy link
Copy Markdown
Author

This PR was marked stale due to lack of activity. It will be closed in 14 days.

LMK if there's something I can do to get this reviewed

@jaronoff97 jaronoff97 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please read the CONTRIBUTING.md file for instructions on how to make a change to the chart, then i'll happily review. thank you!

@github-actions github-actions Bot removed the Stale label Mar 4, 2026
@kema-dev

kema-dev commented Mar 5, 2026

Copy link
Copy Markdown
Author

please read the CONTRIBUTING.md file for instructions on how to make a change to the chart, then i'll happily review. thank you!

Hello, I'm sorry I missed the version bump, this is now fixed

priorityClassName: {{ $collector.priorityClassName }}
{{- end }}
{{- if $collector.trafficDistribution }}
trafficDistribution: {{ $collector.trafficDistribution }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll need to update the json schema to be allowed to set trafficDistribution in the .Values.collectors section.

@github-actions

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Mar 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions Bot closed this Apr 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants