Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/757#discussion_r139250975
--- 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 --
Good catch here. This was a problem someone was going to run into
eventually. Do we also need a check for hdfsClientHosts?
zookeeperClientHosts? Now that I think about it we also need to ensure REST is
installed on the same host as parsers, enrichment and indexing (for topologoy
start/stop). Add a check for metronParsersHost might satisfy all those
indirectly.
---
