You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by justinleet <gi...@git.apache.org> on 2016/11/11 14:59:12 UTC
[GitHub] incubator-metron pull request #356: METRON-563: Ambari Metron service uses i...
GitHub user justinleet opened a pull request:
https://github.com/apache/incubator-metron/pull/356
METRON-563: Ambari Metron service uses incorrect port for installing Elasticsearch templates
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/justinleet/incubator-metron METRON-563
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-metron/pull/356.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #356
----
commit c5ebc08780ec3a73efe544449eb588296c99f287
Author: justinjleet <ju...@gmail.com>
Date: 2016-11-11T14:56:44Z
METRON-563: Ambari Metron service uses incorrect port for installing Elasticsearch templates
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #356: METRON-563: Ambari Metron service uses incorrec...
Posted by dlyle65535 <gi...@git.apache.org>.
Github user dlyle65535 commented on the issue:
https://github.com/apache/incubator-metron/pull/356
+1 great work, @justinleet
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #356: METRON-563: Ambari Metron service uses incorrec...
Posted by justinleet <gi...@git.apache.org>.
Github user justinleet commented on the issue:
https://github.com/apache/incubator-metron/pull/356
PR splits the es_url into 3 fields: es_hosts, es_binary_port, and es_http_port. The two port fields are defaulted to ES's defaults, so only hosts needs to be provided by the user on install.
As a note, only the first entry in es_hosts is used for the installed templates. It was already broken in the case of multiple entries anyway (because it tried to use the old es_url as a whole, but could be a comma delimited list), and I didn't want to put in logic around iterating through servers until one is found and/or retries etc. It seems like a reasonable requirement to have working servers for something that should be done pretty soon after install.
To test this, I spun up an Ambari cluster as detailed https://www.evernote.com/shard/s530/sh/c5551fbd-0ac1-4861-89ce-9c5e37065c52/b13e05f39eaac1a6, along with a Docker instance of Elasticsearch to hit. The new configs all show up in the UI and the ports are populated correctly. The service action to install templates works, and data passes through to indexing using the Bro topology.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request #356: METRON-563: Ambari Metron service uses i...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/356
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---