Properly lock chart dependencies #326

2022-06-11T10:53:16Z

justusbunsi commented

2022-06-11 10:53:16 +00:00

(Migrated from gitea.com)

As mentioned in the Bitnami chart issue, they've created a git tag to
represent the latest full content index.yaml. This should be used
instead of a commit that could be changed.

https://github.com/bitnami/charts/issues/10539#issuecomment-1152641771

I've also removed the Chart.lock file from the ignore list. This file
ensures correct dependency retrieval when using helm dependency build.

https://helm.sh/docs/helm/helm_dependency_build/#helm-dependency-build

Signed-off-by: justusbunsi sk.bunsenbrenner@gmail.com

As mentioned in the Bitnami chart issue, they've created a git tag to represent the latest full content `index.yaml`. This should be used instead of a commit that could be changed. https://github.com/bitnami/charts/issues/10539#issuecomment-1152641771 I've also removed the `Chart.lock` file from the ignore list. This file ensures correct dependency retrieval when using `helm dependency build`. https://helm.sh/docs/helm/helm_dependency_build/#helm-dependency-build Signed-off-by: justusbunsi <sk.bunsenbrenner@gmail.com>

techknowlogick (Migrated from gitea.com) approved these changes 2022-06-12 19:35:48 +00:00

luhahn (Migrated from gitea.com) approved these changes 2022-06-15 07:34:42 +00:00

luhahn (Migrated from gitea.com) left a comment

LGTM

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: lunny/helm-chart#326