Looks like a corresponding change is needed in the unit test. These mock return values should include the full line of output with version instead of only the number:
https://github.com/apache/ambari/blob/44436e1d5e0d28e07285c7b506cf596748ce85fc/ambari-server/src/test/python/TestAmbariServer.py#L3153 https://github.com/apache/ambari/blob/44436e1d5e0d28e07285c7b506cf596748ce85fc/ambari-server/src/test/python/TestAmbariServer.py#L3162 eg. ``` p.communicate.return_value = ('java version "1.7.0_191"', None) ``` and ``` p.communicate.return_value = ('openjdk version "1.8.0_161"', None) ``` [ Full content available at: https://github.com/apache/ambari/pull/2427 ] This message was relayed via gitbox.apache.org for devnull@infra.apache.org
