----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28037/#review61437 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On Nov. 14, 2014, 1:06 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28037/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2014, 1:06 p.m.) > > > Review request for Ambari and Dmitro Lisnichenko. > > > Bugs: AMBARI-8333 > https://issues.apache.org/jira/browse/AMBARI-8333 > > > Repository: ambari > > > Description > ------- > > STR: > 1\. Start ambari-agent > 2\. Make sure it's registered with ambari-server. > 3\. Execute ambari-agent restart > > > > root@u1201:~# ambari-agent restart > Restarting ambari-agent > Verifying Python version compatibility... > Using python /usr/bin/python2.7 > Found ambari-agent PID: 8240 > Stopping ambari-agent > Removing PID file at /var/run/ambari-agent/ambari-agent.pid > ambari-agent successfully stopped > Verifying Python version compatibility... > Using python /usr/bin/python2.7 > Checking for previously running Ambari Agent... > Starting ambari-agent > Verifying ambari-agent process status... > ERROR: ambari-agent start failed. For more details, see > /var/log/ambari-agent/ambari-agent.out: > ==================== > Failed to start ping port listener of: Could not open port 8670 because > port already used by another process: > UID PID PPID C STIME TTY TIME CMD > root 9186 8274 0 11:24 pts/0 00:00:00 apt-cache showpkg > iso-codes > ==================== > Agent out at: /var/log/ambari-agent/ambari-agent.out > Agent log at: /var/log/ambari-agent/ambari-agent.log > > > The reason for this, is that we don't kill child processes, when killing > ambari-agent, during restart. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/main.py 5d33ca4 > ambari-agent/src/test/python/ambari_agent/TestMain.py 7b1a8c8 > > Diff: https://reviews.apache.org/r/28037/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
