-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pin Docker Image and Dependency Versions in Helm Chart #71
Comments
Note: This can cause unexpected outages with no changes on our side, and it generally against the nature of how you deploy services into Kubernetes which should remain static until an engineer decides to upgrade it. Please fix urgently. |
Thanks for raising this! @Steffen911 are there planned changes related to this as part of v1 of this chart? |
@lucas-koontz @AndrewFarley Regarding the sub-charts: We trust bitnami to adhere to semantic versioning and have good experience with that in the past. An option could be that we include the Chart.lock file to realize the pinning, but I'm not sure how that is consumed by tools like Terraform if they are used to deploy. And my understanding is that the chart dependencies are only resolved and updated on helm changes, i.e. they wouldn't cause any "non-interaction" outages. |
@Steffen911 I'm not sure you're understanding what you're doing. You're effectively using "latest" and thus forcing your latest updates upon people not upon their own choosing. I believe it is important to note that this is effectively forcing others to run bleeding edge and not allowing them to choose when they are ready to update. This approach can cause downtime at unexpected times, which I've experienced multiple times first hand, including with your chart. What you're doing is also a large security issue, in that you could push a bad update to the latest 3 tag which compromises the security or integrity if your supply chain gets compromised. This is generally why many who practice secure supply chains have registries configured for No other helm chart of any service I'm aware of uses this anti-pattern of yours. I ask you to go check every single other helm chart out there, All of these and every Helm Chart I've ever seen use static immutable image references to specific images per published chart version, they do not use tags which refer to changing references like What you've done here is a neat trick for inside a company where you can choose to on behalf of your entire company to run bleeding edge. But, you can't and shouldn't do this on behalf of the entire open source community. I ask, or rather beg you to reconsider. On behalf of another passionate open-source contributor who authors and supports numerous other Helm Charts. Do the right thing here. |
Description
The current Langfuse Helm chart does not pin an exact Docker image version, instead using a version that varies with each release (values.yaml#L6). This behavior is similar to using
latest
, which is not explicit enough and can lead to unexpected errors for users relying on the Helm chart.The image tag should be pinned to a specific version. As of today (February 18, 2025), the latest version is
3.28.3
.Additionally, dependencies in
Chart.yaml
are currently soft-locked to major versions (e.g.,16.x.x
for PostgreSQL,7.x.x
for ClickHouse, etc.). This can also introduce unexpected breaking changes. Pinning dependencies to exact versions would improve stability and predictability.Steps to Reproduce:
Expected Behavior:
3.28.3
).Chart.yaml
should be locked to exact versions instead of usingx.x.x
ranges.Additional Context:
This change would improve stability and ensure that users always deploy a known, tested configuration.
The text was updated successfully, but these errors were encountered: