adutra commented on code in PR #1944: URL: https://github.com/apache/polaris/pull/1944#discussion_r2167500715
########## helm/polaris/Chart.yaml: ########## @@ -21,8 +21,8 @@ apiVersion: v2 name: polaris description: A Helm chart for Apache Polaris (incubating) type: application -version: 0.1.0 -appVersion: 1.0.0-incubating-SNAPSHOT +version: 1.0.0 +appVersion: 1.0.0-incubating Review Comment: There are pros and cons: * `latest` is very convenient, but at the same time, not very precise. * Specifying a fixed version in `image.tag`, on the contrary, has the merit of making it clear that the Helm chart works best for that specific tag. I am personally in favor of using `image.tag = latest`, because if people also install the latest version of the helm chart, and provided that we publish images and charts at every release, then they should automatically get a latest chart version that works well with the latest docker image version. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
