You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "Bloodyludi (via GitHub)" <gi...@apache.org> on 2023/09/11 10:40:29 UTC
[GitHub] [pulsar-helm-chart] Bloodyludi opened a new pull request, #393: Add support for configuration of priorityClassName
Bloodyludi opened a new pull request, #393:
URL: https://github.com/apache/pulsar-helm-chart/pull/393
### Motivation
Allow users to configure custom pod priority and preemption by adding the option to set `priorityClassName`.
https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/
### Modifications
Added `priorityClassName` to all deployment and statefulset templates.
### Verifying this change
- [ ] Make sure that the change passes the CI checks.
--
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@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add support for configuration of priorityClassName [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #393:
URL: https://github.com/apache/pulsar-helm-chart/pull/393#issuecomment-4097930718
@scmi-c8y Please rebase the changes since there are conflicts. It might be easier to redo the changes since there are a lot of merge conflicts.
--
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@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add support for configuration of priorityClassName [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #393:
URL: https://github.com/apache/pulsar-helm-chart/pull/393#issuecomment-1849863290
@Bloodyludi Please provide more context of why a user would want to twek pod priority and preemption? Does it solve a problem with Pulsar?
--
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@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add support for configuration of priorityClassName [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #393:
URL: https://github.com/apache/pulsar-helm-chart/pull/393#issuecomment-4371197464
rebased in #685
--
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@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add support for configuration of priorityClassName [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari closed pull request #393: Add support for configuration of priorityClassName
URL: https://github.com/apache/pulsar-helm-chart/pull/393
--
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@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add support for configuration of priorityClassName [pulsar-helm-chart]
Posted by "scmi-c8y (via GitHub)" <gi...@apache.org>.
scmi-c8y commented on PR #393:
URL: https://github.com/apache/pulsar-helm-chart/pull/393#issuecomment-3738926403
Hi, I'm not the OP but our use case is that we define some components of our application (including Pulsar) as "critical", because the application will not operate without them. Setting a priority class would allow us to prioritise scheduling of Pulsar (and other critical component) pods ahead pf optional components and make our deployments more robust during upgrades and configuration changes. So, not really a "Pulsar problem" but more about prioritising some important components ahead of others. We would certainly welcome a change like this.
--
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@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org