[GitHub] metron pull request #1134: METRON-1696: Create the HDFS directory for pcap s...

2018-07-26 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1134 METRON-1696: Create the HDFS directory for pcap sequence files and add required privileges to metron user ## Contributor Comments PCAP parser fails to write pacap sequence file to hdfs on

[GitHub] metron pull request #1136: METRON-1701: Update steps to run the pycapa on ke...

2018-07-31 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1136 METRON-1701: Update steps to run the pycapa on kerberized cluster. ## Contributor Comments Updated the steps to run Pycapa with Kerberos ## Pull Request Checklist Thank you

[GitHub] metron pull request #1136: METRON-1701: Update steps to run the pycapa on ke...

2018-08-01 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/1136#discussion_r206862766 --- Diff: metron-sensors/pycapa/README.md --- @@ -239,25 +239,47 @@ The probe can be used in a Kerberized environment. Follow these additional step

[GitHub] metron pull request #1136: METRON-1701: Update steps to run the pycapa on ke...

2018-08-01 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/1136#discussion_r206863111 --- Diff: metron-sensors/pycapa/README.md --- @@ -239,25 +239,47 @@ The probe can be used in a Kerberized environment. Follow these additional step

[GitHub] metron pull request #1136: METRON-1701: Update steps to run the pycapa on ke...

2018-08-01 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/1136#discussion_r206863799 --- Diff: metron-sensors/pycapa/README.md --- @@ -239,25 +239,47 @@ The probe can be used in a Kerberized environment. Follow these additional step

[GitHub] metron pull request #1136: METRON-1701: Update steps to run the pycapa on ke...

2018-08-01 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/1136#discussion_r206864051 --- Diff: metron-sensors/pycapa/README.md --- @@ -54,8 +54,8 @@ General notes on the installation of Pycapa. ``` export PREFIX=/usr

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-02 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 @mmiklavc can you please floor this PR, I have create follow-on Jira [METRON‌-1709 ](https://issues.apache.org/jira/browse/METRON-1709). ---

[GitHub] metron pull request #1141: Metron-1727: metron-rest service needs to be rest...

2018-08-07 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1141 Metron-1727: metron-rest service needs to be restarted after enabling x-pack for Elastic search. ## Contributor Comments metron-rest service needs to be restarted after enabling x-pack for

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-08 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 Thanks for pointing this out @merrimanr, Yes this will overwrite the the #1019 . If the feature branch is not getting to master sooner then we can keep this change and rebase the feature branch

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-10 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 @mmiklavc I see the 'storm' user as the owner of pcap topology in non-kerberized setup, where as the 'metron' as the owner in kerberized setup. IMHO The pcap should have 

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-14 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 Non Kerberized pcap topology runs under 'storm' which requires write under the 'hadoop' group so we should provide mode 0775 when the cluster is un secure and mode 0755 when t

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-15 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 I am fine wuth closing this PR as this is currently implemented in REST. When pcap gets its own start/stop script in Ambari we can move the code from REST to respective file. ---

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-29 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 closing as it addressed in METRON-1738 ---

[GitHub] metron pull request #1134: METRON-1696: Create the HDFS directory for pcap s...

2018-08-29 Thread MohanDV
Github user MohanDV closed the pull request at: https://github.com/apache/metron/pull/1134 ---

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-29 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 Closed as it is addressed in METRON-1738 ---

[GitHub] metron pull request #1201: METRON-1709 : Added the required scripts to start...

2018-09-18 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1201 METRON-1709 : Added the required scripts to start / stop PCAP topology from Ambari ## Contributor Comments Created PCAP topology as Metron component, Created required scripts to create

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

2018-09-19 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1201 @nickwallen It's my bad I dint run this up in Full Dev, I verified my scripts on a 12 node cluster. I did the required changes to support the Full dev deploymwnt and updated the pull request. ---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

2018-09-20 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1201 @nickwallen having separate panel for PCAP configs is been addressed in the pull request [1132](https://github.com/apache/metron/pull/1132). It will be a duplicate effort here. ---

[GitHub] metron pull request #1206: METRON-1786: Pcap Topology Status Incorrect

2018-09-20 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1206 METRON-1786: Pcap Topology Status Incorrect ## Contributor Comments Added missing params which caused the incorrect PCAP topology status in Ambari. ## Pull Request Checklist

[GitHub] metron pull request #1207: METRON-1695: Expose pcap properties through Ambar...

2018-09-21 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/1207#discussion_r219503222 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/params/params_linux.py

[GitHub] metron pull request #1207: METRON-1695: Expose pcap properties through Ambar...

2018-09-21 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/1207#discussion_r219501331 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/params/params_linux.py

[GitHub] metron issue #1207: METRON-1695: Expose pcap properties through Ambari

2018-09-27 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1207 PCAP related properties needs to be taken out from the [rest_application](https://github.com/apache/metron/blob/master/metron-interface/metron-rest/src/main/config/rest_application.yml#L57). Also

[GitHub] metron issue #1202: METRON-1782 Add Kafka Partition and Offset to Profiler D...

2018-09-28 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1202 +1 Verified on the full dev , below are the debug log from profiler `2018-09-28 10:17:36.465 o.a.m.p.b.ProfileSplitterBolt Thread-6-splitterBolt-executor[7 7] [DEBUG] Received message; topic

[GitHub] metron issue #1205: METRON-1785 Automate deployment of packet capture for de...

2018-10-01 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1205 Deployed a full dev , I executed set of instructions as in Docs `vagrant up vagrant --ansible-tags="pcap" provision` #Stopped the Parser, Enrichment, Indexing, an

[GitHub] metron issue #1207: METRON-1695: Expose pcap properties through Ambari

2018-10-04 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1207 > Cool, thanks for your opinion @nickwallen . I will wait for @MohanDV 's thoughts as well before merging. +1 Thanks @anandsubbu ---

[GitHub] metron pull request #1227: METRON-1807: Auto populate the recommended values...

2018-10-05 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1227 METRON-1807: Auto populate the recommended values to some of the metron config parameters ## Contributor Comments Auto populate the recommended values to below metron config parameters

[GitHub] metron pull request #968: METRON-1497: Rest endpoint '/api/v1/search/search'...

2018-03-19 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/968 METRON-1497: Rest endpoint '/api/v1/search/search' needs to handle null when elastic search response return null for getAggregations ## Contributor Comments Handle null value wh

[GitHub] metron pull request #992: METRON-1522: Fix the typo errors at profiler debug...

2018-04-12 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/992 METRON-1522: Fix the typo errors at profiler debugger readme ## Contributor Comments Fix the typo errors at profiler debugger readme to keep the profile name consistent in flush profile section

[GitHub] metron issue #942: METRON-1461: Modify the MIN, MAX Stellar methods to take ...

2018-04-24 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/942 If we've reached consensus can I get +1 for this ? ---

[GitHub] metron issue #942: METRON-1461: Modify the MIN, MAX Stellar methods to take ...

2018-05-08 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/942 Thanks @cestella . Can you please merge this pull request. ---

[GitHub] metron pull request #1035: METRON-1581: kill the profiler topology immediate...

2018-05-28 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1035 METRON-1581: kill the profiler topology immediately before restarting the topology if it is in 'KILLED' state for longer time ## Contributor Comments Restarting profiler topology f

[GitHub] metron pull request #1052: METRON-1604 : Add power pc to OS family for the H...

2018-06-07 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1052 METRON-1604 : Add power pc to OS family for the HCP management pack repo info ## Contributor Comments Add power pc to OS family for the HCP management pack repo info ## Pull Request

[GitHub] metron pull request #1086: METRON-1645: Check wether the Solr management pac...

2018-06-27 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1086 METRON-1645: Check wether the Solr management pack is installed before configuring the solr principal name. ## Contributor Comments Check wether the Solr management pack is installed