You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/02/07 14:56:34 UTC

[GitHub] [pulsar-helm-chart] elangelo opened a new pull request #225: allow specifying the nodeSelector for the init jobs

elangelo opened a new pull request #225:
URL: https://github.com/apache/pulsar-helm-chart/pull/225


   Fixes #<xyz>
   
   ### Motivation
   
   When deploying pulsar to an AKS cluster with windows nodepools i was unable to specify that the Jobs of the initalize release had to run on linux nodes. With the change i can now specify a node selector for the init jobs.
   
   ### Modifications
   
   add nodeSelector on pulsar_init and bookie_init
   
   ### 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: dev-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar-helm-chart] lhotari commented on pull request #225: allow specifying the nodeSelector for the init jobs

Posted by GitBox <gi...@apache.org>.
lhotari commented on pull request #225:
URL: https://github.com/apache/pulsar-helm-chart/pull/225#issuecomment-1033539128


   > what do i do for the bookkeeper_cluster_initalize jobs? specify another nodeSelector value? or use the pulsar_metadata.nodeSelector value there as well?
   
   I think it's fine to use the pulsar_metadata.nodeSelector value there as well.


-- 
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: dev-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar-helm-chart] elangelo commented on pull request #225: allow specifying the nodeSelector for the init jobs

Posted by GitBox <gi...@apache.org>.
elangelo commented on pull request #225:
URL: https://github.com/apache/pulsar-helm-chart/pull/225#issuecomment-1032828708


   > Thanks for the PR. Would it be possible to use the key `pulsar_metadata.nodeSelector` instead of `initialize_jobs.nodeSelector`? Please also add a placeholder in the default values.yaml with a comment documenting the setting. The value should be empty.
   
   what do i do for the bookkeeper_cluster_initalize jobs? specify another nodeSelector value? or use the pulsar_metadata.nodeSelector value there as well? 


-- 
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: dev-unsubscribe@pulsar.apache.org

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