-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Helm installation uses latest
tag by default
#17540
Comments
Yes, this is an issue we faced as well. I noticed that the Superset version was frequently getting updated whenever the pods restarted. |
Expiriencing the same |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue |
It's still actual |
Same issue with ECS Setup (upgrade to 1.5.0) |
Experiencing the same issue |
Is this still recent? At least by looking at the current helm templates it should default to the appVersion of the chart:
|
Can anyone confirm if this is still a current issue? |
Looks like @cberge908 is correct, there's the line he linked to which points to appVersion here, that gets bumped manually but consistently after official releases: https://github.com/apache/superset/blob/master/helm/superset/Chart.yaml#L18 Currently 3.1.0. I'm going to close this as fixed. |
Using
latest
tag in helm releases is really dangerous story. I've faced the issue today, because new latest tag expects to have 'dashboards.certifiedBy" column in the table which leads to dashboards fetching error.It's much more better to manage app version tag using standard helm approach.
How to reproduce the bug
Expected results
Works
Actual results
Dashboards fetching fails
Screenshots
If applicable, add screenshots to help explain your problem.
![image](https://user-images.githubusercontent.com/91125723/143283495-e5f804aa-fcc9-48d8-a8d2-162cd583ef5c.png)
The text was updated successfully, but these errors were encountered: