You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/08/01 03:49:26 UTC

[GitHub] [pulsar] yapxue opened a new issue, #16895: pulsar bookkeeper.conf cannot cover all bookkeeper config property keys

yapxue opened a new issue, #16895:
URL: https://github.com/apache/pulsar/issues/16895

   **Is your enhancement request related to a problem? Please describe.**
   bookkeeper.conf in https://github.com/apache/pulsar cannot cover all keys in bk_server.conf in https://github.com/apache/bookkeeper . Starting a bookkeeper in pular archive cannot customize some bookkeeper configations by add configmap key values. For example serverNumIOThreads.
   
   **Describe the solution you'd like**
   copy all property keys in bk_server.conf to bookkeeper.conf
   
   **Describe alternatives you've considered**
   A clear and concise description of any alternative solutions or features you've considered.
   
   **Additional context**
   Add any other context or screenshots about the feature request here.
   


-- 
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.apache.org

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


[GitHub] [pulsar] github-actions[bot] commented on issue #16895: pulsar bookkeeper.conf cannot cover all bookkeeper config property keys

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on issue #16895:
URL: https://github.com/apache/pulsar/issues/16895#issuecomment-1234990082

   The issue had no activity for 30 days, mark with Stale label.


-- 
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


[GitHub] [pulsar] yapxue commented on issue #16895: pulsar bookkeeper.conf cannot cover all bookkeeper config property keys

Posted by GitBox <gi...@apache.org>.
yapxue commented on issue #16895:
URL: https://github.com/apache/pulsar/issues/16895#issuecomment-1201988744

   > there's a script just for this purpose: https://github.com/apache/pulsar/blob/master/docker/pulsar/scripts/apply-config-from-env.py You can supply any custom config via env then run this script like `apply-config-from-env.py bookkeeper.conf` To supply config key that's not already in bookkeeper.conf, add prefix `PULSAR_PREFIX_` to it so it can be properly recognized by the script.
   
   But some bookkeeper config like serverNumIOThreads is not in https://github.com/apache/pulsar/blob/master/conf/bookkeeper.conf and not in "keys" in the python script https://github.com/apache/pulsar/blob/master/docker/pulsar/scripts/apply-config-from-env.py#L72 and it cannot be applied.


-- 
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


[GitHub] [pulsar] yapxue closed issue #16895: pulsar bookkeeper.conf cannot cover all bookkeeper config property keys

Posted by GitBox <gi...@apache.org>.
yapxue closed issue #16895: pulsar bookkeeper.conf cannot cover all bookkeeper config property keys
URL: https://github.com/apache/pulsar/issues/16895


-- 
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


[GitHub] [pulsar] MarvinCai commented on issue #16895: pulsar bookkeeper.conf cannot cover all bookkeeper config property keys

Posted by GitBox <gi...@apache.org>.
MarvinCai commented on issue #16895:
URL: https://github.com/apache/pulsar/issues/16895#issuecomment-1201299352

   there's a script just for this purpose: https://github.com/apache/pulsar/blob/master/docker/pulsar/scripts/apply-config-from-env.py
   You can supply any custom config via env then run this script like `apply-config-from-env.py bookkeeper.conf`
   To supply config key that's not already in bookkeeper.conf, add prefix `PULSAR_PREFIX_` to it so it can be properly recognized by the script.


-- 
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