GitHub user mmiklavc opened a pull request:
https://github.com/apache/metron/pull/904
METRON-1400: Elasticsearch service check fails in Ambari
## Contributor Comments
Addresses https://issues.apache.org/jira/browse/METRON-1400
This file hasn't been touched since at least 2016, but the file=sys.stdout
arg to the Logging statement is causing errors. I removed it and changed the ES
host:port info to pull from the Ambari configs as we probably wanted all along.
The regex is similar to what we did to setup links for ES health and indexes
under ES Ambari Quicklinks.
## Testing
### Basic Test
1. Spin up full dev
2. Open Ambari at http://node1:8080
3. Click on Elasticsearch in the service list
4. Verify ES is running. Start it if it is not running.
5. Click "Service Actions" and choose "Run Service Check" from the dropdown
6. The service check should pass
### Check ES http port
1. Click on Elasticsearch in the service list
2. Select "Configs" in the tabs across the top of the page
3. Enter "http_port" in the filter box
4. Change the default "9200-9300" value to something else, e.g. "9700-9800"
5. Click save and restart ES
6. Once ES comes up again, click "Service Actions" and choose "Run Service
Check" again from the dropdown
7. The service check should still pass
## Pull Request Checklist
### For all changes:
- [x] Is there a JIRA ticket associated with this PR? If not one needs to
be created at [Metron
Jira](https://issues.apache.org/jira/browse/METRON/?selectedTab=com.atlassian.jira.jira-projects-plugin:summary-panel).
- [x] Does your PR title start with METRON-XXXX where XXXX is the JIRA
number you are trying to resolve? Pay particular attention to the hyphen "-"
character.
- [x] Has your PR been rebased against the latest commit within the target
branch (typically master)?
### For code changes:
- [x] Have you included steps to reproduce the behavior or problem that is
being changed or addressed?
- [x] Have you included steps or a guide to how the change may be verified
and tested manually?
- [x] Have you ensured that the full suite of tests and checks have been
executed in the root metron folder via:
```
mvn -q clean integration-test install && build_utils/verify_licenses.sh
```
- [x] Have you verified the basic functionality of the build by building
and running locally with Vagrant full-dev environment or the equivalent?
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mmiklavc/metron METRON-1400
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/metron/pull/904.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 #904
----
commit 49aa0bb7c0c6b8ccc5fdbb58592a20c992b4579a
Author: Michael Miklavcic <michael.miklavcic@...>
Date: 2018-01-23T18:34:34Z
Elasticsearch service check fails in Ambari
----
---
