[GitHub] [pulsar-helm-chart] filipecaixeta opened a new issue #250: make proxy httpNumThreads configurable

[GitBox <gi...@apache.org>]

filipecaixeta opened a new issue #250:
URL: https://github.com/apache/pulsar-helm-chart/issues/250


   **Is your feature request related to a problem? Please describe.**
   `httpNumThreads` is hardcoded to 8 in `charts/pulsar/templates/proxy-configmap.yaml`
   When trying to override in values.yaml by using `proxy.configData.httpNumThreads` we get an error because the keys get duplicated
   ```shell
   Error: map[string]interface {}(nil): yaml: unmarshal errors:
     line 44: mapping key "httpNumThreads" already defined at line 34
   xargs: sh: exited with status 255; aborting
   ```
   
   **Describe the solution you'd like**
   Removing the key from `charts/pulsar/templates/proxy-configmap.yaml` and adding it to the values yaml solves the problem. Something like this
   ```yaml
     configData:
       PULSAR_MEM: >
         -Xms64m -Xmx64m -XX:MaxDirectMemorySize=64m
       PULSAR_GC: >
         -XX:+UseG1GC
         -XX:MaxGCPauseMillis=10
         -Dio.netty.leakDetectionLevel=disabled
         -Dio.netty.recycler.linkCapacity=1024
         -XX:+ParallelRefProcEnabled
         -XX:+UnlockExperimentalVMOptions
         -XX:+DoEscapeAnalysis
         -XX:ParallelGCThreads=4
         -XX:ConcGCThreads=4
         -XX:G1NewSizePercent=50
         -XX:+DisableExplicitGC
         -XX:-ResizePLAB
         -XX:+ExitOnOutOfMemoryError
         -XX:+PerfDisableSharedMem
       httpNumThreads: "8"
   ```
   
   **Describe alternatives you've considered**
   Couldn't find any
   


-- 
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] filipecaixeta commented on issue #250: make proxy httpNumThreads configurable

[GitBox <gi...@apache.org>]

filipecaixeta commented on issue #250:
URL: https://github.com/apache/pulsar-helm-chart/issues/250#issuecomment-1078963665


   @lhotari can you review this for me?


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