[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-09 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r173398330 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/metron_service.py

[GitHub] metron issue #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-09 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/946 @mmiklavc yes, correct, like with ES you need to install x-pack for Kibana. You will be prompted for username password after restart of Kibana. ---

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-09 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r173397551 --- Diff: pom.xml --- @@ -97,7 +97,7 @@ ${base_hadoop_version} ${base_hbase_version} ${base_flume_version

[GitHub] metron issue #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-02 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/946 Steps to test: - build centos vagrant dev build box - vagrant ssh - sudo /usr/share/elasticsearch/bin/elasticsearch-plugin install x-pack - sudo /usr/share/elasticsearch/bin/x

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171599970 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/utils/ElasticsearchUtils.java --- @@ -111,19 +111,24

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread wardbekker
GitHub user wardbekker opened a pull request: https://github.com/apache/metron/pull/946 METRON-1465:Support for Elasticsearch X-pack ## Contributor Comments [Please place any comments here. A description of the problem/enhancement, how to reproduce the issue, your testing

[GitHub] metron issue #939: [BUG-96727] xpack support (for discussion only)

2018-02-19 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/939 ok, used the wrong id ---

[GitHub] metron pull request #939: [BUG-96727] xpack support (for discussion only)

2018-02-19 Thread wardbekker
Github user wardbekker closed the pull request at: https://github.com/apache/metron/pull/939 ---

[GitHub] metron pull request #939: [BUG-96727] xpack support (for discussion only)

2018-02-18 Thread wardbekker
GitHub user wardbekker opened a pull request: https://github.com/apache/metron/pull/939 [BUG-96727] xpack support (for discussion only) ## Contributor Comments Hotfix for BUG-96727. Please don't pull. For discussion only ## Pull Request Checklist

[GitHub] metron issue #619: METRON-939 Elasticsearch ES5 with Xshield client support

2018-02-10 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/619 agreed @nickwallen ---

[GitHub] metron pull request #619: METRON-939 Elasticsearch ES5 with Xshield client s...

2018-02-10 Thread wardbekker
Github user wardbekker closed the pull request at: https://github.com/apache/metron/pull/619 ---

[GitHub] metron pull request #930: METRON-1318 updated MacOS instructions and explain...

2018-02-07 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/930#discussion_r166752716 --- Diff: metron-deployment/amazon-ec2/README.md --- @@ -41,11 +50,13 @@ Any platform that supports these tools is suitable, but the following instructio

[GitHub] metron pull request #930: METRON-1318 updated MacOS instructions and explain...

2018-02-07 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/930#discussion_r166752027 --- Diff: metron-deployment/amazon-ec2/README.md --- @@ -41,11 +50,13 @@ Any platform that supports these tools is suitable, but the following instructio

[GitHub] metron pull request #930: METRON-1318 updated MacOS instructions and explain...

2018-02-07 Thread wardbekker
GitHub user wardbekker opened a pull request: https://github.com/apache/metron/pull/930 METRON-1318 updated MacOS instructions and explain AWS defaults ## Contributor Comments updated MacOS instructions and explain AWS defaults, also covers METRON-1318 for MacOS

[GitHub] metron issue #619: METRON-939 Elasticsearch ES5 with Xshield client support

2017-07-05 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/619 @JonZeolla done. --- 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

[GitHub] metron issue #619: METRON-939 Elasticsearch ES5 with Xshield client support

2017-06-27 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/619 hey @cestella, @simonellistonball, see updated contributor notes. It's not ready for a official pull request, but this gives a good idea on the impact on the code for a working

[GitHub] metron issue #619: METRON-939 Elasticsearch ES5 with Xshield client support

2017-06-19 Thread wardbekker
Github user wardbekker commented on the issue: https://github.com/apache/metron/pull/619 Thanks for the review @cestella I've pushed a commit fixing the reported issues. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] metron pull request #619: METRON-939 Elasticsearch ES5 with Xshield client s...

2017-06-19 Thread wardbekker
GitHub user wardbekker opened a pull request: https://github.com/apache/metron/pull/619 METRON-939 Elasticsearch ES5 with Xshield client support ## Contributor Comments This is a work-in-progress fix for Elasticsearch ES5 with Xshield client support. I'm opening a pull re