[GitHub] metron issue #620: Metron-988: UI for viewing alerts generated by Metron

2017-07-27 Thread iraghumitra
Github user iraghumitra commented on the issue: https://github.com/apache/metron/pull/620 @mraliagha The rest interface for retrieving elastic data is on the way I would suggest we should handle it over there. If you have any simpler way to accomplish it in UI plz feel free to

[GitHub] metron pull request #620: Metron-988: UI for viewing alerts generated by Met...

2017-07-27 Thread iraghumitra
Github user iraghumitra commented on a diff in the pull request: https://github.com/apache/metron/pull/620#discussion_r129769257 --- Diff: metron-interface/metron-alerts/README.md --- @@ -0,0 +1,80 @@ +- [Caveats](#caveats) +- [Prerequisites](#prerequisites) +-

[GitHub] metron issue #620: Metron-988: UI for viewing alerts generated by Metron

2017-07-27 Thread iraghumitra
Github user iraghumitra commented on the issue: https://github.com/apache/metron/pull/620 @nickwallen The e2e tests are working fine i checked them again you need to do the below two setps in two consoles. ``` metron-interface/metron-alerts » ./scripts/start-server-for-e2e.sh

[GitHub] metron issue #620: Metron-988: UI for viewing alerts generated by Metron

2017-07-27 Thread mraliagha
Github user mraliagha commented on the issue: https://github.com/apache/metron/pull/620 @iraghumitra Have you found any way to manage multiple Elasticsearch endpoints for load-balancing? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826679 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826587 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826535 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

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

2017-07-27 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/metron/pull/619 Seems like this is failing on some un-related temporary test failures. Can we get Travis kicked, and see what's left to do on this? --- If your project is set up for it, you can reply to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826380 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129824553 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827293 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827680 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129824590 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826663 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826699 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826628 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129825779 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129825816 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827745 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```code - To have a single lib, some things that were in other nifi libraries had to be relocated. ./bundles: ExtensionManager.java-> MODIFIED

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827115 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827277 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827184 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #673: METRON-1069: Ambari MPack documentation around dev...

2017-07-27 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/673 METRON-1069: Ambari MPack documentation around development strategies ## Contributor Comments First cut at documentation around actually making changes to the management pack. I haven't

[GitHub] metron issue #673: METRON-1069: Ambari MPack documentation around developmen...

2017-07-27 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/673 We also might want to include suggestions for editing metron_theme.json. That has become more important now that we are organizing properties in different sections with different widgets Happy

[GitHub] metron pull request #673: METRON-1069: Ambari MPack documentation around dev...

2017-07-27 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/673#discussion_r129865954 --- Diff: metron-deployment/packaging/ambari/README.md --- @@ -0,0 +1,86 @@ +# Ambari Management Pack Development +Typically, Ambari Management Pack

[GitHub] metron pull request #673: METRON-1069: Ambari MPack documentation around dev...

2017-07-27 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/673#discussion_r129871944 --- Diff: metron-deployment/packaging/ambari/README.md --- @@ -0,0 +1,86 @@ +# Ambari Management Pack Development +Typically, Ambari Management Pack

[GitHub] metron issue #673: METRON-1069: Ambari MPack documentation around developmen...

2017-07-27 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/673 @merrimanr I honestly don't recall the steps to make sure we don't break anything when redeploying the mpack. Would you be willing / able to do a quick PR on to my branch, or just give the short

[GitHub] metron pull request #673: METRON-1069: Ambari MPack documentation around dev...

2017-07-27 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/673#discussion_r129867889 --- Diff: metron-deployment/packaging/ambari/README.md --- @@ -0,0 +1,86 @@ +# Ambari Management Pack Development +Typically, Ambari Management

[GitHub] metron issue #673: METRON-1069: Ambari MPack documentation around developmen...

2017-07-27 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/673 Also, I'd like to see a small section about best practices that people should consider when they're adding properties, such as: * If you're adding a new table or topic based on your property,

[GitHub] metron pull request #673: METRON-1069: Ambari MPack documentation around dev...

2017-07-27 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/673#discussion_r129871149 --- Diff: metron-deployment/packaging/ambari/README.md --- @@ -0,0 +1,86 @@ +# Ambari Management Pack Development +Typically, Ambari Management

[GitHub] metron issue #673: METRON-1069: Ambari MPack documentation around developmen...

2017-07-27 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/673 Sure I'll write something up on the mpack redeploy. We could explain metron_theme.json as a seperate PR but these instructions may be incomplete without it. For example, I'm not sure

[GitHub] metron issue #673: METRON-1069: Ambari MPack documentation around developmen...

2017-07-27 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/673 @cestella Can you take a look and let me know if the section on the Jinja templates I added is the sort of concrete example you're looking for? I walked through an existing property, from

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129908796 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/StormStatusServiceImpl.java --- @@ -17,10 +17,7 @@ */

Re: [DISCUSS] Easing the ramp-up into contributing

2017-07-27 Thread Otto Fowler
Beyond this, I think we need to encourage PR review and testing as a way to contribute, along with documentation. So things that help with those topics will be good. Maybe an expended "ways to get involved including”, and why they are important. Increased participation leading to PR’s that don’t

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129897123 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/StormStatusServiceImpl.java --- @@ -17,10 +17,7 @@ */

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129904369 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/StormStatusServiceImpl.java --- @@ -17,10 +17,7 @@ */

Re: [GitHub] metron issue #673: METRON-1069: Ambari MPack documentation around developmen...

2017-07-27 Thread Otto Fowler
This is indeed awesome. What is also missing from my person experience and pov. is how to add a new rpm or extend the functionality. If you don’t add the rpm to the configuration, it won’t get deployed. How do you call Zookeeper? What get’s called when in the scripts? What where the steps

Facing error whith mvn clean install

2017-07-27 Thread bharath phatak
Hi All, I followed the instructions mentioned https://cwiki.apache.org/confluence/display/METRON/Metron+Development+Environment+Setup+Instructions clone git repository and opened the project in intelIj. While running "mvn -DskipTests clean install" , i am getting below error. [ERROR] Failed to

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129920992 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/SupervisorStatus.java --- @@ -0,0 +1,127 @@ +/** + *

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/metron/pull/530 @ottobackwards , can you please give me a similar file mapping for the "test" side of bundles-lib, ie bundles-lib/src/test/java/org/apache/metron/ vs

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/metron/pull/530 @ottobackwards I didn't steal _that_ much of it. Still it shouldn't cause a problem, as long as we get all the updates since this PR started ported in. That one has moved a bit from things

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 @mattf-horton I have reviewed the changes between what I have and 1.3.0, and while there are some one or two slight differences, there is nothing there that functionally improves or applies to

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129923353 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16 @@

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129933909 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I'm actually going to debug this and verify about the dependency --- 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

Re: Facing error whith mvn clean install

2017-07-27 Thread Michael Miklavcic
Hi Bharath, Thanks for the interest in Metron! Is this continuing to be a problem for you? It looks like the maven repo hosting that jar may have been temporarily unavailable. I just tried the URL locally and it appears to be working. Downloaded ->

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129946569 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16 @@

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129920967 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/SupervisorStatus.java --- @@ -0,0 +1,127 @@ +/** + *

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 > My question was, since it throws an exception if no Bundle dependency is stated, if I have a Bundle that is not actually dependent on any other bundles, what should I declare it is dependent

Re: [DISCUSS] Easing the ramp-up into contributing

2017-07-27 Thread Matt Foley
Really good discussion thread, thanks for opening it Justin. I’m a fan of adding javadocs to the publicly available doc set. It’s not the most important of the items listed below, but it is easy, and will push people to be more attentive to dev documentation. METRON-759 is open for that, and

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129932703 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129926344 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/SupervisorStatus.java --- @@ -0,0 +1,127 @@ +/** + *

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129929213 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16 @@

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129930004 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16 @@

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129941180 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```bash . ├── java │   └── org │   └── apache │   └── metron │   ├── bundles │  

Re: Facing error whith mvn clean install

2017-07-27 Thread bharath phatak
Hi Michael, I was able pass on with the above issue.Its resolved. I am facing issue with metron-config [ERROR] 'cp' is not recognized as an internal or external command, [ERROR] operable program or batch file. [ERROR] [ERROR] npm ERR! Windows_NT 10.0.10586 [ERROR] npm ERR! argv

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 WRT: Required Bundle dependency for BundleProvidedDependenciesMojo. That mojo is not run automatically, either in metron or in the nifi case. It is *explicitly* run to determine the

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 No, there is no problem with you referring back to nifi @simonellistonball --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] metron issue #607: METRON-982 add new rest api for storm supervisor status

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/607 @merrimanr we all set? --- 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

[GitHub] metron pull request #642: METRON-984 Stellar functions to decode encoded fie...

2017-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/642 --- 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 is