Re: [PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-25 Thread via GitHub
xiangfu0 merged PR #15356: URL: https://github.com/apache/pinot/pull/15356 -- 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-unsubscr...@pinot.apa

Re: [PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-25 Thread via GitHub
Jackie-Jiang commented on PR #15356: URL: https://github.com/apache/pinot/pull/15356#issuecomment-2752013601 I'm not familiar with how helm works. @xiangfu0 Can you help take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-25 Thread via GitHub
PovilasV1 commented on code in PR #15356: URL: https://github.com/apache/pinot/pull/15356#discussion_r2011527829 ## helm/pinot/values.yaml: ## @@ -90,6 +90,10 @@ pinotAuth: # -- controller: name: c

Re: [PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-25 Thread via GitHub
PovilasV1 commented on code in PR #15356: URL: https://github.com/apache/pinot/pull/15356#discussion_r2011480079 ## helm/pinot/templates/broker/ingress-v1.yaml: ## @@ -20,7 +20,7 @@ {{- if .Values.broker.ingress.v1.enabled }} {{- $ingressPath := .Values.broker.ingress.v1.path

Re: [PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-24 Thread via GitHub
Jackie-Jiang commented on code in PR #15356: URL: https://github.com/apache/pinot/pull/15356#discussion_r2010900795 ## helm/pinot/templates/broker/ingress-v1.yaml: ## @@ -20,7 +20,7 @@ {{- if .Values.broker.ingress.v1.enabled }} {{- $ingressPath := .Values.broker.ingress.v1.pa

Re: [PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-24 Thread via GitHub
codecov-commenter commented on PR #15356: URL: https://github.com/apache/pinot/pull/15356#issuecomment-2748363223 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15356?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] HELM: Add TLS-only support and publish 0.3.3 Chart version [pinot]

2025-03-24 Thread via GitHub
PovilasV1 opened a new pull request, #15356: URL: https://github.com/apache/pinot/pull/15356 Reference issue - https://github.com/apache/pinot/issues/10091 This PR improves the Helm chart's TLS configuration capabilities by: Adding configurable port settings for all components (