Skip to content

Conversation

@CharlieDixon
Copy link
Contributor

@CharlieDixon CharlieDixon commented Mar 11, 2025

Adds a helper function to allow the checksum/secrets annotation to be updated in response to changes in both the default secret.yaml file, as well as any files which match the wildcard patterns *secrets.yaml.encrypted or *secrets.enc.yaml, both of which are often used in conjunction with SOPS.

Resolves #81


@CharlieDixon
Copy link
Contributor Author

Tested locally and is working as expected.

To replicate, install the Helm chart to a cluster, passing in an additional secrets manifest which matches the wildcard syntax included in these changes. For example:

helm install . -f  /path/to/lightdash-secrets.yaml.encrypted --generate-name -n lightdash`.

The checksum/secrets value will look something like this:

Annotations: checksum/config: 3d43a82693a770f503798e4474b0025d484b6111738d56049a26318fff32ad46
             checksum/secrets: 078bf8c86f91c1624faafbf55382d1a021f95fbd051616130bbc0644cf217db7

Next, make an arbitrary change to the content of either the default secrets.yaml or, in the example above,lightdash-secrets.yaml.encrypted values and perform an upgrade:

helm upgrade [CHART] . -f /path/to/lightdash-secrets.encrypted -n lightdash

N.B. The checksum generated for the ConfigMap remains the same but the secrets checksum has been updated in response to the changes which triggers a rolling restart of the deployment:

Annotations: checksum/config: 3d43a82693a770f503798e4474b0025d484b6111738d56049a26318fff32ad46
             checksum/secrets: 69b15bbe794142ea854bb7c04ae3f6ec71c263c038df5426c54caf076faea202

@rephus rephus added this to the External contributions milestone Mar 11, 2025
@CharlieDixon CharlieDixon marked this pull request as draft October 20, 2025 15:14
@CharlieDixon
Copy link
Contributor Author

Set to draft pending resolution of conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📝 To-Do

Development

Successfully merging this pull request may close these issues.

Include encryped secrets manifests syntax to checksum updates

2 participants