[GitHub] incubator-metron pull request #540: METRON-869 Include build instructions fo...

2017-04-21 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/incubator-metron/pull/540 --- 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 #678: METRON-1076: Management UI status check always reports th...

2017-08-03 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/678 +1 (non-binding) Validated that with this fix, Ambari displays the correct status as opposed to the earlier behavior. --- If your project is set up for it, you can reply to this email

[GitHub] metron issue #688: METRON-1094: MaaS will not start due to classpath error r...

2017-08-10 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/688 I'm +1 (non-binding) Fired up a full dev and was able to test that `maas_service.sh` starts up fine. No errors seen in the YARN application logs. A few observations though

[GitHub] metron issue #680: DO NOT MERGE: METRON-1078: Metron Indexing fails to stop ...

2017-08-06 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/680 +1 (non-binding). Validated the latest fix to be working great. Thank you, @dlyle65535 . --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] metron issue #680: DO NOT MERGE: METRON-1078: Metron Indexing fails to stop ...

2017-08-04 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/680 Sounds great, @dlyle65535 ! --- 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

[GitHub] metron issue #680: DO NOT MERGE: METRON-1078: Metron Indexing fails to stop ...

2017-08-04 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/680 Hi @dlyle65535 can it be a two step validation as follows: a) We first verify if we are able to hit the storm URL, if that fails -> then we mark the service as stopped. b) If the st

[GitHub] incubator-metron pull request #583: METRON-949 Change from error to warning ...

2017-05-12 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/incubator-metron/pull/583 METRON-949 Change from error to warning message for Metron REST in Ambari service_advisor ## Contributor Comments Reduced the level to WARN in service_advisor.py. Steps

[GitHub] metron issue #583: METRON-949 Change from error to warning message for Metro...

2017-05-17 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/583 Could I get some love for this PR? :-) --- 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

[GitHub] incubator-metron pull request #583: METRON-949 Change from error to warning ...

2017-05-12 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/incubator-metron/pull/583 --- 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] incubator-metron pull request #583: METRON-949 Change from error to warning ...

2017-05-12 Thread anandsubbu
GitHub user anandsubbu reopened a pull request: https://github.com/apache/incubator-metron/pull/583 METRON-949 Change from error to warning message for Metron REST in Ambari service_advisor ## Contributor Comments Reduced the level to WARN in service_advisor.py. Steps

[GitHub] metron issue #583: METRON-949 Change from error to warning message for Metro...

2017-05-25 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/583 Thank you, @merrimanr . Can you please help merge this pull request? --- 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

[GitHub] metron issue #583: METRON-949 Change from error to warning message for Metro...

2017-05-19 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/583 @merrimanr , this is because of the following reason: In the "Assign Masters" page of the Ambari deployment wizard, there is a check for the Metron REST host to have storm

[GitHub] metron issue #583: METRON-949 Change from error to warning message for Metro...

2017-05-23 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/583 @merrimanr, let me know if the above comment answers your question. --- 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

[GitHub] metron issue #604: METRON-978: Management UI init script uses hard-coded MET...

2017-06-05 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/604 +1 (non-binding) since this ensures that the Metron UI service starts up properly --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] metron issue #759: METRON-1187 Indexing/Profiler Kafka ACL Groups Not Setup ...

2017-09-15 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/759 +1 (non-binding) Indeed, this was a tricky one. Thank you @nickwallen for your root cause and fix!! I spun up a 12 node cluster with this patch, kerberized the cluster and confirmed

[GitHub] metron pull request #723: METRON-1141: NameError: metronProfilerHost is not ...

2017-08-30 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/723 METRON-1141: NameError: metronProfilerHost is not defined in stack_advisor.py ## Contributor Comments ** Testing Done ** - Modified `service_advisor.py` on the Ambari host with the mpack

[GitHub] metron issue #731: METRON-1146 Refactor for ParserExtensionConfig and Rest g...

2017-09-06 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/731 Hi @ottobackwards , I guess you wanted to name the PR title as METRON-1136 instead of METRON-1146? ---

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r138362796 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -506,29 +506,27 @@ public

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r138362926 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -550,4 +548,90 @@ else

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r138362754 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -550,4 +548,90 @@ else

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r138362605 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -550,4 +548,90 @@ else

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r138362883 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -550,4 +548,90 @@ else

[GitHub] metron issue #727: METRON-1146: Add ability to parse JSON string into JSONOb...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/727 Hi @nickwallen thank you. Sure, I see value in simplifying the naming. How does `TO_JSON_OBJECT`, `TO_JSON_MAP` and `TO_JSON_LIST` sound? ---

[GitHub] metron issue #727: METRON-1146: Add ability to parse JSON string into JSONOb...

2017-09-12 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/727 Thanks very much @ottobackwards ! @nickwallen , pushed latest commit with the functions renamed. Please have a look. ---

[GitHub] metron pull request #719: METRON-1137: Build RPM for Metron MaaS as a part o...

2017-08-29 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/719 METRON-1137: Build RPM for Metron MaaS as a part of rpm-docker packaging ## Contributor Comments Fixed metron.spec and the pom.xml to now copy and generate metron-maas-service RPM

[GitHub] metron pull request #714: METRON-1128: MAAS_GET_ENDPOINT - Unable to resolve...

2017-08-29 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/714#discussion_r135785927 --- Diff: metron-analytics/metron-maas-service/README.md --- @@ -126,16 +126,20 @@ Now let's install some prerequisites: Start Squid via `service

[GitHub] metron pull request #714: METRON-1128: MAAS_GET_ENDPOINT - Unable to resolve...

2017-08-29 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/714#discussion_r135786862 --- Diff: metron-analytics/metron-maas-service/README.md --- @@ -126,16 +126,20 @@ Now let's install some prerequisites: Start Squid via `service

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-05 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r136954740 --- Diff: metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/StringFunctionsTest.java --- @@ -449,4 +450,68 @@ public

[GitHub] metron pull request #727: METRON-1146: Add ability to parse JSON string into...

2017-09-05 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r136954647 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -450,4 +454,40 @@ public

[GitHub] metron issue #723: METRON-1141: NameError: metronProfilerHost is not defined...

2017-08-30 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/723 Closing this pull request, since it is the same as https://github.com/apache/metron/pull/722 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] metron pull request #723: METRON-1141: NameError: metronProfilerHost is not ...

2017-08-30 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/metron/pull/723 --- 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

[GitHub] metron pull request #807: METRON-1263: Start Alerts UI service after Metron ...

2017-10-19 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/807 METRON-1263: Start Alerts UI service after Metron REST ## Contributor Comments Currently the Alerts UI service starts up before any of the Metron services does. Ideally, it should be started

[GitHub] metron issue #841: METRON-1316 Fastcapa Fails to Compile in Test Environment

2017-11-22 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/841 +1 (non-binding), btw ---

[GitHub] metron issue #841: METRON-1316 Fastcapa Fails to Compile in Test Environment

2017-11-22 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/841 As always, very crisp @nickwallen ! My +1 (non-binding) still holds. ---

[GitHub] metron pull request #846: METRON-1328: Enhance platform-info.sh script to ch...

2017-11-22 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/846 METRON-1328: Enhance platform-info.sh script to check if docker daemon is running ## Contributor Comments Run the `docker info` command along with checking the version. **Case

[GitHub] metron issue #846: METRON-1328: Enhance platform-info.sh script to check if ...

2017-11-22 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/846 Hi @ottobackwards , thanks for your review. I prefixed a 'WARNING' to the message text so it would catch the eye of the user. Also, I removed the else bit for docker running scenario

[GitHub] metron issue #852: METRON-1239 Drop extra dev environments

2017-11-28 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/852 Hi @nickwallen , I found one more reference to quick dev - [here](https://github.com/nickwallen/metron/blob/7145b06fff9f4ecc8e63d9cdd56ed955a49e676d/metron-interface/metron-rest/src/test/resources

[GitHub] metron issue #852: METRON-1239 Drop extra dev environments

2017-11-29 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/852 > I was confused about that README.vm file. Any idea what that is for? I am not entirely sure, @nickwallen . I could find a lot of similarities between this README.vm and the main Met

[GitHub] metron issue #852: METRON-1239 Drop extra dev environments

2017-11-30 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/852 +1 (non-binding) @nickwallen . This is a much needed fix since it is now straight-forward to anyone new and wanting to try Metron. ---

[GitHub] metron pull request #850: METRON-1335: Install metron-maas-service RPM as a ...

2017-11-27 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/850 METRON-1335: Install metron-maas-service RPM as a part of the full-dev deployment ## Contributor Comments Modified mpack metainfo.xml to install metron-maas-service RPM as well

[GitHub] metron issue #850: METRON-1335: Install metron-maas-service RPM as a part of...

2017-12-04 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/850 bump. Any feedback on this pull request? ---

[GitHub] metron issue #839: METRON-1311 Service Check Should Check Elasticsearch Inde...

2017-11-10 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/839 +1 (non-binding) works fine, and a much needed fix as well ! If for some reason, we were not able to install the templates during the startup, it would be caught by the service check

[GitHub] metron issue #839: METRON-1311 Service Check Should Check Elasticsearch Inde...

2017-11-11 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/839 +1 (non-binding). Validated again on full-dev. Works fine, @nickwallen ! ---

[GitHub] metron issue #841: METRON-1316 Fastcapa Fails to Compile in Test Environment

2017-11-20 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/841 Hi @nickwallen > I am not sure exactly what the problem is, but the same condition occurs in master. I would call this a pre-existing condition that we can handle with a separate

[GitHub] metron issue #841: METRON-1316 Fastcapa Fails to Compile in Test Environment

2017-11-15 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/841 Hi @nickwallen , tested on the vagrant fastcapa deployment. Deployed fine, and I was able to see this message as well: ``` TASK [debug

[GitHub] metron issue #829: METRON-1296 Full Dev Fails to Deploy Index Templates

2017-11-07 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/829 @nickwallen , for the record full dev spun up fine with the fix. Thank you! ---

[GitHub] metron issue #829: METRON-1296 Full Dev Fails to Deploy Index Templates

2017-11-07 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/829 Hi @nickwallen , thank you for the fix. It is my bad, I should have added the line `commands = IndexingCommands(params)` before the `try` block in my previous fix :(. I am under way

[GitHub] metron issue #850: METRON-1335: Install metron-maas-service RPM as a part of...

2017-12-05 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/850 Sure, that would be great. ---

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-28 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 You're right @nickwallen , it wouldn't work. I reworked on the patch and here are the latest changes: - moved the ES template installation back to the Indexing "start" method -

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-29 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 If the install fails the first time due to ES being down, then we WARN and move on. If ES service comes up later, the admin needs to install the templates subsequently by using 'Ambari -> Met

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-29 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 I have added this to the Elasticsearch README in my recent commit. Please see here: https://github.com/apache/metron/pull/817/files#diff-325aea0d364d12c8637eef347ebcfca6

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147681110 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT "http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147708307 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT "http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147699187 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT "http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147714561 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT "http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147737598 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -86,8 +86,13 @@ rm ${SENSOR}.template The stock set of Elasticsearch templates

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-29 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 Hi @ottobackwards , let me know what you think. ---

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-29 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147587694 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py

[GitHub] metron issue #864: METRON-1348 Metron Service Checks Use Wrong Hostname

2017-12-20 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/864 I did another 12-node deployment on Centos 7 with this PR (bypassed the kafka issue by installing Kafka broker on all nodes). The fix worked just perfect. Thanks much @nickwallen ! +1

[GitHub] metron pull request #807: METRON-1263: Start Alerts UI service after Metron ...

2017-10-19 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/807#discussion_r145701351 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/addon-services/METRON/CURRENT/role_command_order.json --- @@ -13,16 +13,18

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-27 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 Restarting travis ---

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-27 Thread anandsubbu
GitHub user anandsubbu reopened a pull request: https://github.com/apache/metron/pull/817 METRON-1283: Install Elasticsearch template as a part of the mpack startup scripts ## Contributor Comments For a Metron multi-node deployment using mpack, the Elasticsearch template

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-27 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 Hi @ottobackwards , made the change to remove the ES template installation from 'load_web_template' role for vagrant deployments. Fired up full-dev with the change and it comes up fine. Was able

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-27 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/metron/pull/817 ---

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-26 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/817 METRON-1283: Install Elasticsearch template as a part of the mpack startup scripts ## Contributor Comments For a Metron multi-node deployment using mpack, the Elasticsearch template

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-04 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/888 METRON-1389: Zeppelin notebook import does not work with Ambari 2.6 ## Contributor Comments In recent versions of Ambari, [Shiro Authentication](https://zeppelin.apache.org/docs/0.6.0

[GitHub] metron pull request #869: METRON-1362 Improve Metron Deployment README

2018-01-03 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/869#discussion_r159430163 --- Diff: metron-deployment/packaging/ambari/metron-mpack/README.md --- @@ -0,0 +1,127 @@ + + +This provides a Management Pack (MPack

[GitHub] metron pull request #869: METRON-1362 Improve Metron Deployment README

2018-01-03 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/869#discussion_r159426522 --- Diff: metron-deployment/packaging/ambari/metron-mpack/README.md --- @@ -0,0 +1,127 @@ + + +This provides a Management Pack (MPack

[GitHub] metron issue #869: METRON-1362 Improve Metron Deployment README

2018-01-03 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/869 +1 (non-binding). Looks good @nickwallen ! ---

[GitHub] metron issue #894: METRON-1326: Metron deploy with Kerberos fails on Ambari ...

2018-01-18 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/894 +1 thank you @mmiklavc. Spun up a 12-node Centos 7 cluster with the latest fix and validated the following: * Did 'Stop All Services' -> no issues seen with ES and Kibana servi

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-16 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/888#discussion_r161735380 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-22 Thread anandsubbu
GitHub user anandsubbu reopened a pull request: https://github.com/apache/metron/pull/888 METRON-1389: Zeppelin notebook import does not work with Ambari 2.6 ## Contributor Comments In recent versions of Ambari, [Shiro Authentication](https://zeppelin.apache.org/docs/0.6.0

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-22 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/metron/pull/888 ---

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-22 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/888#discussion_r163022818 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-16 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/888#discussion_r161955419 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py

[GitHub] metron issue #894: METRON-1326: Metron deploy with Kerberos fails on Ambari ...

2018-01-13 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/894 > I'd like to treat this as distinct from the status check if that seems reasonable to you. I have to admit, I didn't even know the service check piece existed until now. Sure, it ma

[GitHub] metron pull request #894: METRON-1326: Metron deploy with Kerberos fails on ...

2018-01-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/894#discussion_r161185438 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_slave.py

[GitHub] metron pull request #894: METRON-1326: Metron deploy with Kerberos fails on ...

2018-01-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/894#discussion_r161185471 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/KIBANA/5.6.2/package/scripts/kibana_master.py --- @@ -93,7

[GitHub] metron pull request #894: METRON-1326: Metron deploy with Kerberos fails on ...

2018-01-12 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/894#discussion_r161184707 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_master.py

[GitHub] metron issue #907: METRON-1427: Add support for storm 1.1 and hdp 2.6

2018-01-25 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/907 Hi @cestella , I did a 12-node deploy on CentOS 7 with this patch. Post-kerberization, I noticed the following errors in Metron REST. Is this a related issue or a different one

[GitHub] metron issue #888: METRON-1389: Zeppelin notebook import does not work with ...

2018-01-24 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/888 @ottobackwards @cestella and @nickwallen - any other feedback on this PR? The travis failures seems to be in an unrelated piece of code. ---

[GitHub] metron issue #904: METRON-1400: Elasticsearch service check fails in Ambari

2018-01-24 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/904 +1 works fine @mmiklavc . Verified on CentOS 7 multinode cluster. Simulated the problem first and then ran with the fix. Here is a sample output with the fix in place

[GitHub] metron pull request #907: METRON-1427: Add support for storm 1.1 and hdp 2.6

2018-01-24 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/907#discussion_r163600738 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/service_check.py

[GitHub] metron pull request #888: METRON-1389: Zeppelin notebook import does not wor...

2018-01-09 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/888#discussion_r160591890 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py

[GitHub] metron issue #907: METRON-1427: Add support for storm 1.1 and hdp 2.6

2018-01-30 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/907 Hi @cestella , with the latest commit having the normalized producer, I was able to spin up a 12-node Centos 7 cluster, install with HDP 2.6 base and kerberize as well without issues. Ran

[GitHub] metron pull request #935: METRON-1386: Fix Metron Website Required Links

2018-02-12 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/935 METRON-1386: Fix Metron Website Required Links ## Contributor Comments Made the following changes to the Metron ASF site: * Fixed the missing entities and made corrections per the Apache

[GitHub] metron pull request #932: METRON-1451: On Centos full dev, Metron Indexing s...

2018-02-08 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/932 METRON-1451: On Centos full dev, Metron Indexing shows up as stopped ## Contributor Comments **Root Cause** The python-requests module introduced a VendorAlias import machinery

[GitHub] metron pull request #932: METRON-1451: On Centos full dev, Metron Indexing s...

2018-02-08 Thread anandsubbu
GitHub user anandsubbu reopened a pull request: https://github.com/apache/metron/pull/932 METRON-1451: On Centos full dev, Metron Indexing shows up as stopped ## Contributor Comments **Root Cause** The python-requests module introduced a VendorAlias import machinery

[GitHub] metron pull request #932: METRON-1451: On Centos full dev, Metron Indexing s...

2018-02-08 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/metron/pull/932 ---

[GitHub] metron pull request #938: METRON-1457: Move ASF links to main page in the Me...

2018-02-16 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/938 METRON-1457: Move ASF links to main page in the Metron website ## Contributor Comments Apache Whimsy checks for the site requisites in the main index.html and not inside sub-levels. This PR

[GitHub] metron pull request #869: METRON-1362 Improve Metron Deployment README

2018-01-02 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/869#discussion_r159361598 --- Diff: metron-deployment/README.md --- @@ -15,178 +15,134 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See

[GitHub] metron pull request #869: METRON-1362 Improve Metron Deployment README

2018-01-02 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/869#discussion_r159280034 --- Diff: metron-deployment/README.md --- @@ -15,178 +15,132 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See

[GitHub] metron issue #864: METRON-1348 Metron Service Checks Use Wrong Hostname

2017-12-20 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/864 Hi @nickwallen , I tried this on a 12-node cluster. I validated that `clusterHostInfo` is populated properly for the alerts_ui, management_ui and rest_ui hosts. However, in my case

[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...

2018-02-26 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Thanks @justinleet for the pointer. I have made a change to use the Powered By logo instead of the foundation logo with feather, since I thought this looked better. I also added a direct link

[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...

2018-02-27 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Thanks all. ---

[GitHub] metron issue #943: METRON-1462: Separate ES and Kibana from Metron Mpack

2018-02-28 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/943 +1 Works as advertised. I used both mpacks to deploy a 12-node CentOS 7 cluster (Ambari 2.6.0.0 and HDP 2.6.3.0). Was able to kerberize the cluster and get bro indices into ES. ---

[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...

2018-02-25 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Thank you @mmiklavc and @nickwallen for your reply on the dev list. Any feedback on this PR? ---

[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...

2018-02-27 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Roger that. Here's the new look. Let me know if this looks ok @justinleet and @cestella . ![image](https://user-images.githubusercontent.com/20395490/36736657-323cff0a-1bff-11e8-8d13