Github user justinleet commented on a diff in the pull request:
https://github.com/apache/metron/pull/757#discussion_r139943087
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/service_advisor.py
---
@@ -69,6 +69,10 @@ def getServiceComponentLayoutValidations(self, services,
hosts):
message = "Metron REST must be colocated with an instance of
STORM SUPERVISOR"
items.append({ "type": 'host-component', "level": 'WARN',
"message": message, "component-name": 'METRON_REST', "host": metronRESTHost })
+ if metronRESTHost not in hbaseClientHosts:
--- End diff --
Updated the service advisor to check on metronParsersHost
---
