----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30930/#review72153 -----------------------------------------------------------
Ship it! Ship It! - Vitalyi Brodetskyi On Лют. 12, 2015, 1:10 після полудня, Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30930/ > ----------------------------------------------------------- > > (Updated Лют. 12, 2015, 1:10 після полудня) > > > Review request for Ambari and Vitalyi Brodetskyi. > > > Bugs: AMBARI-9595 > https://issues.apache.org/jira/browse/AMBARI-9595 > > > Repository: ambari > > > Description > ------- > > PROBLEM: > > For example, on CentoOS 6.5, manually install mysql > > [root@hdpsecc01 init.d]# mysql --version > mysql Ver 14.14 Distrib 5.6.20, for Linux (x86_64) using EditLine wrapper > > /etc/init.d/mysqld > shows that when the service is not running, it is > 'MySQL is not running' > > While Ambari 1.7.0 > /var/lib/ambari- > agent/cache/stacks/HDP/2.0.6/services/HIVE/package/scripts/my_services.py > > status_cmd = format('service > > {daemon_name} status | grep running') > > so it thinks the service is running when it is not. > > Jonathan Hurley suggested that a fix needs to fix on other linux distros as it > doesn't work there either > Can keep status_cmd = format('service {daemon_name} > > status') for redhat family OS's, but for other families, need a different > not_if command. > > For now, for centos/redhat can try change the ambari script above > status_cmd = format('service > > {daemon_name} status | grep running') > > to > status_cmd = format('service {daemon_name} > > status') > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/mysql_service.py > f908c75 > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_mysql_server.py > bb1f235 > > Diff: https://reviews.apache.org/r/30930/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
