pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] vli02 opened a new pull request #2484: Fixed the issue 2481 by not to take PULSAR_MEM from env.
Date Thu, 30 Aug 2018 16:29:36 GMT
vli02 opened a new pull request #2484: Fixed the issue 2481 by not to take PULSAR_MEM from
env.
URL: https://github.com/apache/incubator-pulsar/pull/2484
 
 
   ### Motivation
   
   Command `./bin/pulsar-admin clusters list` failed with `Error: Could not find or load main
class "` in my k8s deployment by using helm charts.
   
   ### Modifications
   
   With helm deployment the env var PULSAR_MEM is quoted with `"`, taking it for `pulsar-admin`
will make the exec command to be:
   
   ```exec /docker-java-home/bin/java -cp '/pulsar/conf:::/pulsar/lib/*:' -Dlog4j.configurationFile=log4j2.yaml
-Djava.net.preferIPv4Stack=true '"' -Xms1g -Xmx1g '-XX:MaxDirectMemorySize=1g"' -client -Dio.netty.leakDetectionLevel=disabled
... org.apache.pulsar.admin.cli.PulsarAdminTool /pulsar/conf/client.conf clusters list```
   
   The fix is to take out the use of env var PULSAR_MEM in `conf/pulsar_tools_env.sh`.
   
   ### Result
   
   `pulsar-admin` works as expected.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message