[GitHub] metron issue #711: METRON-1127: Add ability to escalate alerts for external ...

2017-08-31 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/711 > The alert or list of alerts should be annotated with the user who escalated the alert(s) and time the alert(s) was escalated. That JIRA works for me @merrimanr . So the assumption right

[GitHub] metron issue #711: METRON-1127: Add ability to escalate alerts for external ...

2017-08-31 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/711 I create a Jira for this: https://issues.apache.org/jira/browse/METRON-1147. It's could probably use some more description but this is a start. --- If your project is set up for it, you can rep

[GitHub] metron pull request #694: METRON-1085: Add REST endpoint to save a user prof...

2017-08-31 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/694#discussion_r136439615 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/AlertsProfileControllerIntegrationTest.java --- @@ -0,0 +1,226 @@

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

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r136436430 --- 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-08-31 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/727#discussion_r136438190 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java --- @@ -450,4 +454,40 @@ public Obj

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

2017-08-31 Thread anandsubbu
GitHub user anandsubbu opened a pull request: https://github.com/apache/metron/pull/727 METRON-1146: Add ability to parse JSON string into JSONObject for stellar ## Contributor Comments This pull request allows the ability to parse a given JSON string and return a JSON object. H

[GitHub] metron issue #713: METRON-1130 - Updating Amazon EC2 README.md

2017-08-31 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/713 Hm, yeah, I can't seem to assign it to you either @as22323 --- 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 h

[GitHub] metron pull request #726: METRON-1145: Profiler mpack does not create kafka ...

2017-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/726 --- 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 enab

[GitHub] metron issue #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/690 I have refactored this to be stellar centric. We will create a second archive for the stand alone shell from stellar-common. I also documented it in the stellar README.md. The q

[GitHub] metron pull request #717: METRON-1134: Allow parser command line options to ...

2017-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/717 --- 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 enab

[GitHub] metron pull request #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread ottobackwards
GitHub user ottobackwards reopened a pull request: https://github.com/apache/metron/pull/690 METRON-1091 Package STELLAR shell as stand alone This PR adds metron-deployment/packaging/archive, and the metron-stellar-shell module. This module packages a stellar shell environment t

[GitHub] metron issue #717: METRON-1134: Allow parser command line options to be spec...

2017-08-31 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/717 Looks good +1 --- 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

[GitHub] metron issue #726: METRON-1145: Profiler mpack does not create kafka acl and...

2017-08-31 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/726 Tested this in full dev with Kerberos enabled. The profiler topology starts up without issue and data appears in HBase after creating a profile. +1 --- If your project is set up for it, you can

Re: [DISCUSS] Metron release 0.4.1

2017-08-31 Thread Matt Foley
Please mark them 0.4.1, as that’s what the community says we want to call the upcoming release, and everything that’s there when I throw the switch will be included. Jon and Anand, will they be in by end/day Friday? Thanks, --Matt On 8/31/17, 7:45 AM, "Nick Allen" wrote: Matt, et al - For

[GitHub] metron pull request #725: METRON-1136 : Track Master

2017-08-31 Thread ottobackwards
Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/725 --- 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 issue #725: METRON-1136 : Track Master

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/725 so, the commits don't close pr's on branches I guess?? --- 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 h

[GitHub] metron issue #717: METRON-1134: Allow parser command line options to be spec...

2017-08-31 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/717 Ok, added the workers and ackers in the README. Take a look and let me know if you see anything else. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] metron pull request #717: METRON-1134: Allow parser command line options to ...

2017-08-31 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/717#discussion_r136409849 --- Diff: metron-platform/metron-parsers/README.md --- @@ -103,6 +103,15 @@ then it is assumed to be a regex and will match any topic matching the pattern (

[GitHub] metron issue #712: METRON-1129 Management UI - Package Node Dependencies in ...

2017-08-31 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/712 I think suppressing the warnings is good enough for now. --- 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 hav

[GitHub] metron issue #726: METRON-1145: Profiler mpack does not create kafka acl and...

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/726 You should add something to the PR Template [ ] Does this require ACL's or something --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] metron issue #702: METRON-1114: Add group by capabilities to search REST end...

2017-08-31 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/702 The latest commit adds the ability to sum a field within each group with the "scoreField". --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: Maven Exec REPL with all Metron Dependencies

2017-08-31 Thread Nick Allen
My use case is a little different than #690. I just want to be able to test things in the REPL during development. I am just trying to find the quickest way to go from code change to executing that code change in the REPL. On Thu, Aug 31, 2017 at 11:02 AM Otto Fowler wrote: > If this is over d

[GitHub] metron pull request #726: METRON-1145: Profiler mpack does not create kafka ...

2017-08-31 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/metron/pull/726 METRON-1145: Profiler mpack does not create kafka acl and hbase acls correctly in kerberized cluster ## Contributor Comments Title says it all. The management pack will not start the profiler i

[GitHub] metron issue #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/690 Right, I think the disconnect is that I would envision a stellar shell with metron stellar functions added as libs, as you suggested. That's why I was thinking we would doc in stellar-common, I mi

[GitHub] metron pull request #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread ottobackwards
Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/690 --- 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 issue #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/690 I'm going to close this and refactor --- 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 e

[GitHub] metron issue #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/690 @JonZeolla I don't want to document it in the stellar-common module. Stellar is being broken out of metron. The packaging of stellar as a shell, with the likely inclusion of metron-* stella

[GitHub] metron issue #690: METRON-1091 Package STELLAR shell as stand alone

2017-08-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/690 bump - any comments on this? Any ideas for deployment? can we leave for later? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

Re: Maven Exec REPL with all Metron Dependencies

2017-08-31 Thread Otto Fowler
If this is over doing it based on your question, I apologize ahead of time. There are a few things here: * the way the stellar functions in metron are package with regards to dependencies and usage * the way the functions are categorized ( ie. should some functions even try to load based on runni

[GitHub] metron issue #725: METRON-1136 : Track Master

2017-08-31 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/725 +1 by inspection, assuming CI build passes. Thanks @ottobackwards --- 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 pull request #721: METRON-1138 Improve Error Message with Bad Profile...

2017-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/721 --- 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 enab

Re: Maven Exec REPL with all Metron Dependencies

2017-08-31 Thread Otto Fowler
So there is https://github.com/apache/metron/pull/690, but that is not what you are asking. The issue is that even if you -pl metron-platform/metron-management it does not bring in all the dependencies ( you are getting FSDataInputStream or some other error right?). I can get both to load by doin

Re: [DISCUSS] Metron release 0.4.1

2017-08-31 Thread Nick Allen
Matt, et al - For JIRAs that are going into master, should we be marking these as "Next + 1" or "0.4.1" ? On Thu, Aug 31, 2017 at 8:17 AM zeo...@gmail.com wrote: > Can I advocate to get METRON-1129 in the RC, and throw in a second vote for > METRON-1134? Both in an attempt to better support of

[GitHub] metron pull request #716: METRON-1132 Enhance Profiler Debug Functions

2017-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/716 --- 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 enab

[GitHub] metron pull request #725: METRON-1136 : Track Master

2017-08-31 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/metron/pull/725 METRON-1136 : Track Master Track master for feature branch. This PR resolves conflicts in the rpm spec and resource copy area of the pom. - [x] Is there a JIRA ticket associated with

Maven Exec REPL with all Metron Dependencies

2017-08-31 Thread Nick Allen
Previously, I found that you could run the REPL from the Metron source by running the following command from the ​source root ​ directory. This is useful because it lets you run the REPL with the libraries that will be available ​during Enrichment.​ mvn exec:java \ -Dexec.mainClass="org.apache.m

[GitHub] metron pull request #704: METRON-1117 Filters Functions Returned by '%functi...

2017-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/704 --- 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 enab

[GitHub] metron issue #721: METRON-1138 Improve Error Message with Bad Profile Expres...

2017-08-31 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/721 +1 by inspection, this is really great. --- 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 ena

[GitHub] metron issue #704: METRON-1117 Filters Functions Returned by '%functions'

2017-08-31 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/704 +1 by inspection, great stuff --- 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 w

[GitHub] metron issue #716: METRON-1132 Enhance Profiler Debug Functions

2017-08-31 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/716 +1 by inspection, this is great stuff @nickwallen --- 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

Re: [DISCUSS] Metron release 0.4.1

2017-08-31 Thread zeo...@gmail.com
Can I advocate to get METRON-1129 in the RC, and throw in a second vote for METRON-1134? Both in an attempt to better support of prod/offline use. Happy to provide testing cycles for the former. Jon On Wed, Aug 30, 2017 at 11:41 AM Anand Subramanian < asubraman...@hortonworks.com> wrote: > Hi M

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

2017-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/719 --- 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 enab

Re: sensor-parser-config-history

2017-08-31 Thread Simon Elliston Ball
That looks like it's the front for when there was versioning of config in the rest api. The versioning was based on hibernate envers so had licence issues, and stored change history into the rest rdbms. That functionality should probably come back one day once we have a proper approach to confi

sensor-parser-config-history

2017-08-31 Thread Otto Fowler
https://github.com/apache/metron/blob/master/metron-interface/metron-config/src/app/model/sensor-parser-config-history.ts Can anyone explain the usage and purpose of this type and how it relates to sensor-parser-config? ottO