[GitHub] metron issue #941: METRON-1355: Convert metron-elasticsearch to new infrastr...

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/941 No, that can wait for the follow-on ticket. Could you add a quick one-sentence blurb about the ElasticsearchTestUtils in the README so things are easier for anyone to test later (with the

[GitHub] metron issue #941: METRON-1355: Convert metron-elasticsearch to new infrastr...

2018-03-06 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/941 Sorry I should have documented this better in the PR description. The docker-machine ip address needs to be substituted in ElasticsearchTestUtils temporarily if you want to run this on your local

[GitHub] metron issue #941: METRON-1355: Convert metron-elasticsearch to new infrastr...

2018-03-05 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/941 I'm unable to get the integration tests running locally. I've been able to get the docker containers up and running, but ES isn't exposed at localhost, only through the explicit docker-machine