You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2022/01/12 13:06:57 UTC

[GitHub] [pinot] WyriHaximus commented on a change in pull request #7997: Add Ingress to Broker and Controller helm charts

WyriHaximus commented on a change in pull request #7997:
URL: https://github.com/apache/pinot/pull/7997#discussion_r783057189



##########
File path: kubernetes/helm/pinot/Chart.yaml
##########
@@ -21,7 +21,7 @@ apiVersion: v1
 appVersion: 0.2.5-SNAPSHOT

Review comment:
       Even though this is supposed to be the application in side this chart's version, does it make sure to match the version field on line 24?




-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org