----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61521/#review182506 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Aug. 9, 2017, 10:35 a.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61521/ > ----------------------------------------------------------- > > (Updated Aug. 9, 2017, 10:35 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Myroslav Papirkovskyy, and > Sid Wagle. > > > Bugs: AMBARI-21688 > https://issues.apache.org/jira/browse/AMBARI-21688 > > > Repository: ambari > > > Description > ------- > > Fix two raceconditions: > First causes agent to hang forever during initial connection: > > > > INFO 2017-08-08 16:55:06,780 transport.py:329 - Starting receiver > loopINFO 2017-08-08 16:55:06,781 transport.py:271 - Sending frame: 'STOMP', > headers={'host': 0, 'accept-version': '1.2'} > > > Second one causes heartbeating fail with: > > > > ERROR 2017-08-08 10:12:08,328 HeartbeatThread.py:167 - Error in > responseId sequence - restarting > INFO 2017-08-08 10:12:08,328 ExitHelper.py:56 - Performing cleanup before > exiting... > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/CommandStatusDict.py d6cbdcc > ambari-agent/src/main/python/ambari_agent/ComponentStatusExecutor.py > be3eb5b > ambari-agent/src/main/python/ambari_agent/HeartbeatThread.py e2fe4af > ambari-agent/src/main/python/ambari_agent/HostStatusReporter.py c45b64a > ambari-agent/src/main/python/ambari_agent/InitializerModule.py 9b031f7 > > ambari-agent/src/main/python/ambari_agent/listeners/ServerResponsesListener.py > 6d23c37 > ambari-agent/src/main/python/ambari_agent/security.py 9509619 > ambari-common/src/main/python/ambari_stomp/adapter/websocket.py 220f399 > ambari-common/src/main/python/ambari_ws4py/websocket.py b5c1fd3 > > > Diff: https://reviews.apache.org/r/61521/diff/1/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
