[GitHub] metron pull request #894: METRON-1326: Metron deploy with Kerberos fails on ...

[anandsubbu]

Github user anandsubbu commented on a diff in the pull request:

    https://github.com/apache/metron/pull/894#discussion_r161185438
  
    --- Diff: 
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_slave.py
 ---
    @@ -56,8 +59,27 @@ def status(self, env):
             import params
             env.set_params(params)
             Logger.info('Check status of Elasticsearch data node')
    -        status_cmd = "service elasticsearch status"
    -        Execute(status_cmd)
    +
    +        # return codes defined by LSB
    +        # 
http://refspecs.linuxbase.org/LSB_3.0.0/LSB-PDA/LSB-PDA/iniscrptact.html
    +        cmd = ('service', 'elasticsearch', 'status')
    +        rc, out = shell.call(cmd, sudo=True, quiet=False)
    +
    +        if rc == 3:
    --- End diff --
    
    Same comment as Kibana


---