Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/799#discussion_r144863737
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/enrichment_commands.py
---
@@ -232,3 +241,48 @@ def set_hbase_acls(self):
Logger.info("Done setting HBase ACLs")
self.set_hbase_acl_configured()
+
+ def service_check(self, env):
+ """
+ Performs a service check for Enrichment.
+ :param env: Environment
+ """
+ Logger.info("Checking for Geo database")
+ metron_service.check_hdfs_file_exists(self.__params,
self.__params.geoip_hdfs_dir + "/GeoLite2-City.mmdb.gz")
--- End diff --
Ok, I see it now under a global properties key "geo.hdfs.file". There is
nothing in Ambari MPack for it, which might complicate using it. I am just
thinking through, if we would need to first introduce it as an Ambari-managed
configuration value.
---
