Skip to content

Add checksum override for beyla config#2555

Merged
grcevski merged 1 commit intomainfrom
skl/beyla-config-roll
Feb 26, 2026
Merged

Add checksum override for beyla config#2555
grcevski merged 1 commit intomainfrom
skl/beyla-config-roll

Conversation

@skl
Copy link
Member

@skl skl commented Feb 26, 2026 •
edited
Loading

Adds a config checksum override for the Beyla DaemonSet so parent charts can drive rollouts when using external configmaps. Updates values/README to document the new option and adds unit tests for override and fallback hashing. Bumps the chart version to 1.12.0.

Tested on a local kind cluster:

# deploy beyla helm chart with default values to default namespace
helm install beyla ./charts/beyla

# flip one default value with default checksum behaviour, confirm restart OK
helm upgrade beyla ./charts/beyla --set "config.data.attributes.kubernetes.enabled=false"

# set custom checksum, confirm restart OK
helm upgrade beyla ./charts/beyla --set "config.checksum=one"

# set same custom checksum, confirm no restart OK
helm upgrade beyla ./charts/beyla --set "config.checksum=one"

# set new custom checksum, confirm restart OK
helm upgrade beyla ./charts/beyla --set "config.checksum=two"

Fixes #2128

@skl skl requested a review from a team as a code owner February 26, 2026 16:33
@skl skl force-pushed the skl/beyla-config-roll branch from d5c72e4 to e509f38 Compare February 26, 2026 16:34
grcevski
grcevski approved these changes Feb 26, 2026
View reviewed changes
Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

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

LGTM!

@grcevski grcevski merged commit 9faa7ab into main Feb 26, 2026
19 checks passed
@grcevski grcevski deleted the skl/beyla-config-roll branch February 26, 2026 17:08
@grcevski
Copy link
Contributor

grcevski commented Feb 26, 2026

I merged, we still need to manually release the helm chart.

@skl skl mentioned this pull request Feb 27, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rafaelroquetto rafaelroquetto rafaelroquetto approved these changes

@grcevski grcevski grcevski approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Implement config reload for helm chart

3 participants

@skl @grcevski @rafaelroquetto