----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60986/#review180991 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/package/scripts/params_linux.py Lines 73-74 (patched) <https://reviews.apache.org/r/60986/#comment256399> Maybe I'm just reading this wrong. Doesn't the if-statement say if the last character is NOT 0-9, meaning it's a letter, then you're adding m onto it? >>> foo = "1024m" >>> if not foo[-1].isdigit(): ... foo = foo + 'm' ... >>> print foo 1024mm - Jonathan Hurley On July 19, 2017, 7:04 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60986/ > ----------------------------------------------------------- > > (Updated July 19, 2017, 7:04 p.m.) > > > Review request for Ambari, Di Li, Jonathan Hurley, Sumit Mohanty, Sid Wagle, > and Tim Thorpe. > > > Bugs: AMBARI-21528 > https://issues.apache.org/jira/browse/AMBARI-21528 > > > Repository: ambari > > > Description > ------- > > Repro Steps: > > * Installed BI 4.2.0 cluster on IBM Ambari 2.2.2 with Zookeeper > * Upgraded Ambari to 2.5.2.0-146 > * Registered HDP 2.6.2.0 repo, installed packages > * Ran service checks > * Started Express Upgrade > > Result: _Service Check ZooKeeper_ step failed with {{KeeperErrorCode = > ConnectionLoss for /zk_smoketest}} > > This was caused by Zookeeper dying immediately during restart: > ``` > Error occurred during initialization of VM > Too small initial heap > ``` > > Before EU > ``` > export JAVA_HOME=/usr/jdk64/java-1.8.0-openjdk-1.8.0.77-0.b03.el7_2.x86_64 > export ZOOKEEPER_HOME=/usr/iop/current/zookeeper-server > export ZOO_LOG_DIR=/var/log/zookeeper > export ZOOPIDFILE=/var/run/zookeeper/zookeeper_server.pid > export SERVER_JVMFLAGS=-Xmx1024m > export JAVA=$JAVA_HOME/bin/java > export CLASSPATH=$CLASSPATH:/usr/share/zookeeper/* > ``` > > After EU > ``` > export JAVA_HOME=/usr/jdk64/java-1.8.0-openjdk-1.8.0.77-0.b03.el7_2.x86_64 > export ZOOKEEPER_HOME=/usr/hdp/current/zookeeper-client > export ZOO_LOG_DIR=/var/log/zookeeper > export ZOOPIDFILE=/var/run/zookeeper/zookeeper_server.pid > export SERVER_JVMFLAGS=-Xmx1024 > export JAVA=$JAVA_HOME/bin/java > ``` > > Note missing "m" in memory setting. > > zookeeper-env template contains, > ``` > export SERVER_JVMFLAGS={{zk_server_heapsize}} > ``` > > In this cluster, zookeeper-env contains, > zk_server_heapsize: "1024" > > While the params_linux.py file has some inconsistencies with appending the > letter "m". > ``` > zk_server_heapsize_value = > str(default('configurations/zookeeper-env/zk_server_heapsize', "1024m")) > zk_server_heapsize = format("-Xmx{zk_server_heapsize_value}") > ``` > > Instead, it should be, > ``` > zk_server_heapsize_value = > str(default('configurations/zookeeper-env/zk_server_heapsize', "1024")) > zk_server_heapsize_value = zk_server_heapsize_value.strip() > if len(zk_server_heapsize_value) > 0 and not > zk_server_heapsize_value[-1].isdigit(): > zk_server_heapsize_value = zk_server_heapsize_value + "m" > zk_server_heapsize = format("-Xmx{zk_server_heapsize_value}") > ``` > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/package/scripts/params_linux.py > 0780d2e > > > Diff: https://reviews.apache.org/r/60986/diff/1/ > > > Testing > ------- > > Python unit tests passed, > > ---------------------------------------------------------------------- > Total run:1161 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >