Skip to content

feat(helm): Add service.trafficDistribution to values.yaml - #7057

Open
agermel wants to merge 1 commit into
grafana:mainfrom
agermel:feat/helm-service-trafficDistribution
Open

agermel wants to merge 1 commit into
grafana:mainfrom
agermel:feat/helm-service-trafficDistribution

Conversation

@agermel

@agermel agermel commented Sep 8, 2026

Copy link
Copy Markdown

Brief description of Pull Request

Adds a service.trafficDistribution value to the Alloy Helm chart. When set, it renders spec.trafficDistribution on the Service, guarded to Kubernetes 1.34+.

Issue(s) fixed by this Pull Request

Fixes #7017

Notes to the Reviewer

This is my first contribution to this project, and I’m really excited to be involved. I’ve enjoyed working on this change and hope to continue contributing to the project in the future.

Thank you for taking the time to review my PR!

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated
  • This pull request was substantially generated with AI assistance (see the GenAI policy)

@agermel
agermel requested a review from a team as a code owner September 8, 2026 14:42
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Signed commits report

1 of 1 commit between main and feat/helm-service-trafficDistribution could not be fully verified:

Commit Author Reason Message
d47ac43e agermel@foxmail.com unknown_key feat(helm): Add service.trafficDistribution to values.yaml

This repository requires all commits to be signed. See GitHub docs on commit signature verification.

@agermel
agermel force-pushed the feat/helm-service-trafficDistribution branch from 815619b to d47ac43 Compare September 8, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

helm: support spec.trafficDistribution on the Alloy Service

1 participant