You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "psheets (via GitHub)" <gi...@apache.org> on 2023/04/04 13:31:52 UTC
[GitHub] [pinot] psheets opened a new issue, #10538: Add an extraContainerPort array option to k8s helm chart
psheets opened a new issue, #10538:
URL: https://github.com/apache/pinot/issues/10538
What needs to be done?
- There should be a way to pass extra configurations for container and service ports when deploying with the helm chart.
Why the feature is needed (e.g. describing the use case).
- The otel service we are using to scrap metrics from the cluster requires the Prometheous metrics port to be configured as a containerPort. There is not currently and option to do that using the values options.
initial idea/proposal
- Mimic the extraEnv implementation
- add
extraContainerPort:
- containerPort: 1234
protocol: FOO
name: BAR
to the values file and append any configurations to the spec.template.spec.container.ports path.
--
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.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
[GitHub] [pinot] psheets commented on issue #10538: Add an extraContainerPort array option to k8s helm chart
Posted by "psheets (via GitHub)" <gi...@apache.org>.
psheets commented on issue #10538:
URL: https://github.com/apache/pinot/issues/10538#issuecomment-1496248240
@abhioncbr I changed the code on our company branch already. Created a PR here: https://github.com/apache/pinot/pull/10540
--
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
[GitHub] [pinot] abhioncbr commented on issue #10538: Add an extraContainerPort array option to k8s helm chart
Posted by "abhioncbr (via GitHub)" <gi...@apache.org>.
abhioncbr commented on issue #10538:
URL: https://github.com/apache/pinot/issues/10538#issuecomment-1496137960
Thanks for reporting.
Please assign this issue to me. Thanks
--
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
Re: [I] Add an extraContainerPort array option to k8s helm chart [pinot]
Posted by "psheets (via GitHub)" <gi...@apache.org>.
psheets closed issue #10538: Add an extraContainerPort array option to k8s helm chart
URL: https://github.com/apache/pinot/issues/10538
--
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