You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "lhotari (via GitHub)" <gi...@apache.org> on 2024/03/22 12:58:35 UTC
[PR] Use podManagementPolicy OrderedReady for Broker sts when Functions are enabled [pulsar-helm-chart]
lhotari opened a new pull request, #474:
URL: https://github.com/apache/pulsar-helm-chart/pull/474
Fixes #473
### Motivation
See #473. Broker startup could fail when Function Worker is enabled in the Broker since it tries to access other unready Brokers.
### Modifications
Use `podManagementPolicy: OrderedReady` for Broker StatefulSet when Functions are enabled.
### 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] Use podManagementPolicy OrderedReady for Broker sts when Functions are enabled [pulsar-helm-chart]
Posted by "Mortom123 (via GitHub)" <gi...@apache.org>.
Mortom123 commented on PR #474:
URL: https://github.com/apache/pulsar-helm-chart/pull/474#issuecomment-2015858233
Great. I encountered crashing brokers trying to access other ones, a while ago but never figured out why!
--
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] Use podManagementPolicy OrderedReady for Broker sts when Functions are enabled [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #474:
URL: https://github.com/apache/pulsar-helm-chart/pull/474#issuecomment-2020762616
podManagementPolicy is immutable on existing statefulsets. I had to add a solution for preserving the previous setting for existing installations so that upgrades don't break.
One solution for changing this for an existing deployment would be to delete the broker statefulset manually and then upgrading the deployment.
--
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] Use podManagementPolicy OrderedReady for Broker sts when Functions are enabled [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #474:
URL: https://github.com/apache/pulsar-helm-chart/pull/474#issuecomment-2015057075
Please review @frankjkelly @Mortom123
--
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] Use podManagementPolicy OrderedReady for Broker sts when Functions are enabled [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari merged PR #474:
URL: https://github.com/apache/pulsar-helm-chart/pull/474
--
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