Re: Additional board report?

2016-11-07 Thread John D. Ament
Yep. So do you want to leave it in or pull it? If you leave it in, I need a mentor to review and sign off on it. John On 2016-11-07 15:24 (-0500), Billie Rinaldi wrote: > James, > > Reports are every 3 months now. Podlings only have to report every month > when they are new. Now Metron repo

[GitHub] incubator-metron pull request #346: METRON-554: Require proper error handlin...

2016-11-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/346 --- 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 featu

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/347 I'll miss those verbose logs. +1 pending the vagrant checks. --- 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: Additional board report?

2016-11-07 Thread Billie Rinaldi
James, Reports are every 3 months now. Podlings only have to report every month when they are new. Now Metron reports are due in January, April, July, and October. Billie On Mon, Nov 7, 2016 at 11:47 AM, James Sirota wrote: > We realized we forgot to submit so I appended it to the existing lis

Re: Additional board report?

2016-11-07 Thread James Sirota
We realized we forgot to submit so I appended it to the existing list of reports. Hope that is ok. Thanks, James 05.11.2016, 07:12, "John D. Ament" : > Dear Metron Podling, > > Thanks for filing an off-cycle report. I was curious to know why you had > submitted it? I don't see any mailing lis

[GitHub] incubator-metron issue #345: METRON-532 Define Profile Period When Calling P...

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/345 @mattf-horton No worries, get better, man. --- 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 thi

[GitHub] incubator-metron issue #345: METRON-532 Define Profile Period When Calling P...

2016-11-07 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/incubator-metron/pull/345 Apologies, but I'm going to be offline for a couple days, due to a surgery. I will fix this when I get back. --Matt --- If your project is set up for it, you can reply to this email

[GitHub] incubator-metron issue #332: METRON-527: Connection pool shut down seen when...

2016-11-07 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/332 I'm satisfied with my PR after further investigation. --- 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

[GitHub] incubator-metron pull request #332: METRON-527: Connection pool shut down se...

2016-11-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/332 --- 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 featu

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/347 Resolved a merge conflict with SnortParserTest and changed Travis configuration to not run mvn install (since we're already doing it as part of our tests). Check out those logs! The sta

[GitHub] incubator-metron issue #338: METRON-295: Script parsing bolt

2016-11-07 Thread rkarthik29
Github user rkarthik29 commented on the issue: https://github.com/apache/incubator-metron/pull/338 The pull request is due to some errors with executing test cases. I am not sure if they are related to the parser, if someone guides me i can look at it. --- If your project is set up f

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/347 @mmiklavc Good question. We definitely need to exclude the SLF4J bindings that we don't use. The most common one is the Log4J2 binding that Storm brings along ("org.apache.logging.log4j

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/347 Ok - you beat me in. I think my PR should be dependent on this. When this goes in, I'll sync up with master etc etc --- If your project is set up for it, you can reply to this ema

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/347 @cestella I don't think that will be an issue since the binding is now coming from the metron-platform pom but I will definitely test it in vagrant after we get some feedback. --- If yo

[GitHub] incubator-metron pull request #336: METRON-210: Decom Multi-Node Vagrant Dep...

2016-11-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/336 --- 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 featu

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/347 @merrimanr Love this effort. We just went through something similar in getting the apache curator versions lined up for the Storm upgrade support ticket. Thinking more about it, I'm not su

Re: Help with custom enrichment / parser

2016-11-07 Thread Tyler Moore
Michael, Thanks for the reply, and to everyone helping y'all rock! I am getting data through the "assets" kafka spout, which is the spout for my parser, but I am not seeing anything coming through indexing spout. I didn't see any logs for the storm supervisor but I attached the enrichment log name

[GitHub] incubator-metron issue #343: METRON-548 Improve Profiler documentation

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/343 +1 by inspection. I like the stats example --- 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 th

[GitHub] incubator-metron pull request #341: METRON-535: Global Validations type stil...

2016-11-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/341 --- 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 featu

[GitHub] incubator-metron issue #332: METRON-527: Connection pool shut down seen when...

2016-11-07 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/332 Please hold off on merging this a bit. Anand and I are looking into an issue that is almost certainly separate, but doing due diligence. --- If your project is set up for it, you can r

[GitHub] incubator-metron issue #347: METRON-541: Set log levels for Metron component...

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/347 Did you check this on vagrant? I seem to remember there being some drama when I tried to remove the slf4j dependencies around some of the data management functions. Specifically, calling

[GitHub] incubator-metron pull request #335: METRON-531: Ensure licenses for bundled ...

2016-11-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/335 --- 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 featu

[GitHub] incubator-metron pull request #320: Metron-402: Snort timestamp field shows ...

2016-11-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/320 --- 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 featu

[GitHub] incubator-metron pull request #347: METRON-541: Set log levels for Metron co...

2016-11-07 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/incubator-metron/pull/347 METRON-541: Set log levels for Metron components This PR accomplishes primarily 2 things: sets the log level for all components (and their dependencies) to ERROR and suppresses ERROR lo

[GitHub] incubator-metron issue #320: Metron-402: Snort timestamp field shows up wron...

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/320 confirmed --- 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 issue #320: Metron-402: Snort timestamp field shows up wron...

2016-11-07 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/320 Just confirming I still have a +1 with the JUnit version change added after the fact --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-metron issue #336: METRON-210: Decom Multi-Node Vagrant Deployment

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/336 +1 by inspection --- 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 wish

[GitHub] incubator-metron issue #335: METRON-531: Ensure licenses for bundled compone...

2016-11-07 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/incubator-metron/pull/335 > Alright, I think I cleared out extraneous mentions in the NOTICEs. How does it look now? Very good. :+1: Big kudos to you @cestella! --- If your project is set up for it, you

[GitHub] incubator-metron issue #346: METRON-554: Require proper error handling when ...

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/346 Can you please assign this to yourself in JIRA and put it in the 0.2.2BETA release? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-metron pull request #346: METRON-554: Require proper error handlin...

2016-11-07 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/346#discussion_r86792656 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/stellar/StellarPredicateProcessor.java --- @@ -51,6 +48,11 @@ publi

[GitHub] incubator-metron pull request #346: METRON-554: Require proper error handlin...

2016-11-07 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/incubator-metron/pull/346 METRON-554: Require proper error handling when invalid input is fed to Threat triage rules This completes https://issues.apache.org/jira/browse/METRON-554 Non-boolean-returning f

[GitHub] incubator-metron issue #343: METRON-548 Improve Profiler documentation

2016-11-07 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/343 Thanks @mattf-horton . I made the changes that you suggested. --- 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 pro

[GitHub] incubator-metron issue #335: METRON-531: Ensure licenses for bundled compone...

2016-11-07 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/335 Alright, I think I cleared out extraneous mentions in the NOTICEs. How does it look now? --- If your project is set up for it, you can reply to this email and have your reply appear on G

Re: [DISCUSS] Next Release Content

2016-11-07 Thread Casey Stella
I agree; it looks limited in scope. I changed the version. Thanks for the contribution, Matt! Casey On Sun, Nov 6, 2016 at 3:22 PM, Matt Foley wrote: > Please consider including > > https://github.com/apache/incubator-metron/pull/345 > > METRON-532 Define Profile Period When Calling

[GitHub] incubator-metron pull request #345: METRON-532 Define Profile Period When Ca...

2016-11-07 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/345#discussion_r86785336 --- Diff: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/GetProfile.java --- @@ -132,35 +144,33 @

Re: Metron with HDP 2.5

2016-11-07 Thread Dima Kovalyov
I have installed Metron on single-node HDP 2.5 using this article: https://community.hortonworks.com/articles/60805/deploying-a-fresh-metron-cluster-using-ambari-serv.html So far it works just fine except ElasticSearch, I am unable to make it work properly on single-node, by default ElasticSearch