[GitHub] incubator-metron pull request #543: METRON-857 Ability to completely build p...

2017-04-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/543 --- 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

Re: [DISCUSS] Modify bylaws to allow speculative branches

2017-04-24 Thread James Sirota
The concrete example where this would be used is the Alerts UI branch that Raghu is currently working on: https://github.com/iraghumitra/metron-alerts It would require a lot of feedback from the community and would need to iterate rapidly. Branch review in this sense would mean that people take

[GitHub] incubator-metron issue #543: METRON-857 Ability to completely build project ...

2017-04-24 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/543 +1 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 does not have this feature enable

[GitHub] incubator-metron issue #535: METRON-859: Use REST application with Kerberos

2017-04-24 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/535 Docs have been updated. --- 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] incubator-metron pull request #545: METRON-883 Capture Bro Plugin Enhancemen...

2017-04-24 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/incubator-metron/pull/545 METRON-883 Capture Bro Plugin Enhancements from bro/bro-plugins There are two versions of the Bro Kafka Plugin. I authored both of these and contributed one to Metron and the othe

[GitHub] incubator-metron pull request #537: METRON-864 fix typo in indexing readme

2017-04-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/537 --- 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 #535: METRON-859: Use REST application with Ke...

2017-04-24 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/incubator-metron/pull/535 --- 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 fe

[GitHub] incubator-metron pull request #535: METRON-859: Use REST application with Ke...

2017-04-24 Thread merrimanr
GitHub user merrimanr reopened a pull request: https://github.com/apache/incubator-metron/pull/535 METRON-859: Use REST application with Kerberos ## Contributor Comments This PR enables the REST application to function in a Kerberized cluster. Testing instructions are as follow

[GitHub] incubator-metron issue #543: METRON-857 Ability to completely build project ...

2017-04-24 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/543 done --- 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 #543: METRON-857 Ability to completely build project ...

2017-04-24 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/543 I wasn't able to spin this up without changing the Dockerfile to be `FROM centos:centos6` instead of `FROM centos:6.6`. This also matches rpm-docker. Any concern with making that change

Re: [DISCUSS] Modify bylaws to allow speculative branches

2017-04-24 Thread Justin Leet
I'd prefer to still keep "Review then Commit" but I'd be interested in lowering the review bar for it as long as the branch gets a solid final review before merge. I don't think a feature branch needs to be micromanaged, but people should still take a quick glance at the commits that come through.

Re: [DISCUSS] Modify bylaws to allow speculative branches

2017-04-24 Thread Michael Miklavcic
"Branch committers ... do not cast binding votes or vetoes in the project." It sounds similar to what we do for other PRs except in this case it's 1..n contributors instead of just one. The step for getting the feature branch merged into trunk sounds reasonable and clear enough, but as Casey mentio

[GitHub] incubator-metron pull request #538: METRON-868 Fix documentation on building...

2017-04-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/538 --- 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

Re: Quick Dev - Atlas Images

2017-04-24 Thread David Lyle
I think it's a really good idea. There is some complexity: a) Image releases do not map 1:1 with Metron releases and Atlas doesn't allow -SNAPSHOT in their release number scheme. That is, we'll have different versions of the image that work with a 0.4.1-SNAPSHOT Metron and some released versions o

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Casey Stella
haha you're not thick; you're noticing a bit of a hack that should probably be corrected. The use of parsers here is entirely to opportunistically take advantage of the fact taht the parsers have as direct dependencies all of the stellar functions in Metron. This was done prior to us being able t

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112982281 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apache Metro

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/544 --- 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

Quick Dev - Atlas Images

2017-04-24 Thread Nick Allen
Right now, we have the images that get pushed to Atlas for Quick Dev versioned independently from the rest of Metron. We currently have versions 0.1.0 and 0.2.0. What happens when a user downloads an official release of Metron, like 0.3.1, and

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112974165 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apache

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/538 +1 from me --- 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 #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @ottobackwards This actually causes issues in just building metron-deployment with mvn package alone (regardless of which target is being run), so it wouldn't help. The purpose

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112972432 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apache Met

[GitHub] incubator-metron pull request #538: METRON-868 Fix documentation on building...

2017-04-24 Thread justinleet
GitHub user justinleet reopened a pull request: https://github.com/apache/incubator-metron/pull/538 METRON-868 Fix documentation on building RPMs ## Contributor Comments Update doc to run install instead of package as a prereq to the RPM build. ## Pull Request Checklist

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 Travis! --- 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 #538: METRON-868 Fix documentation on building...

2017-04-24 Thread justinleet
Github user justinleet closed the pull request at: https://github.com/apache/incubator-metron/pull/538 --- 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 f

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112972023 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apache Met

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112968303 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apa

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/538 Would this be fixed if we just ran build-rpms every time? In other words got rid of the profile flag requirement? --- If your project is set up for it, you can reply to this email a

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/538 also: http://blog.sonatype.com/2008/04/how-to-share-resources-across-projects-in-maven/ --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-metron issue #544: METRON-866 Quickdev and Fulldev readme misses D...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/544 +1 Thanks for the backstop, Nick. I can't believe I left that doggone thing there. --- 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 #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 For posterity: https://issues.apache.org/jira/browse/METRON-868 --- 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 pr

[GitHub] incubator-metron issue #544: METRON-866 Quickdev and Fulldev readme misses D...

2017-04-24 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/544 Done. Please take a look. --- 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 enabl

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 +1 - pulled in the dependency changes, deleted my .m2 cache and the build ran as expected. Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112962800 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112961765 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112961515 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

Re: [GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread Otto Fowler
Sorry to be late to this, but since METRON-777 will have to add to this list, can we get a summary of what we think is going on? On April 24, 2017 at 10:23:57, mmiklavc (g...@git.apache.org) wrote: Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/538

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @mmiklavc Added a quick note, and listed those dependencies off. Let me know if there's anything else you want in there. --- If your project is set up for it, you can reply to this ema

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/538 @justinleet Works for me. Please indicate that we have a dependency across projects that is not shown via the dependency mechanism. Specifically, we have a dependency on the tarballs outpu

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @mmiklavc I made the proposed change and pushed it so it can at least be seen. I'll can easily adjust if you either don't want to do this or have suggestions or anything else. --- If

Re: metron UI

2017-04-24 Thread Justin Leet
To elaborate on Ryan's reply a bit, the UI is a fairly new component in Metron and was merged into master not long ago. It's definitely open to being iterated and improved on, and getting feedback on direction users would like it to go would be a great contribution in and of itself (especially as

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Otto Fowler
OK, not to be thick, I just don’t see where any class from metron-parsers gets called or instantiated. ¯\_(ツ)_/¯ On April 24, 2017 at 09:55:38, Casey Stella (ceste...@gmail.com) wrote: All of the current stellar functions are depended on either directly or indirectly by the parser. On Mon, Apr

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112953201 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

Re: metron UI

2017-04-24 Thread Ryan Merriman
What features would you like to see included? On Mon, Apr 24, 2017 at 8:51 AM, moshe jarusalem wrote: > Hi All, > I have recently run metron UI. I am a bit surprised because it has very few > features such as configuring some ingestions and topologies. > > Have I not configured it properly or it

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Casey Stella
All of the current stellar functions are depended on either directly or indirectly by the parser. On Mon, Apr 24, 2017 at 9:54 AM, Otto Fowler wrote: > too bad there isn’t a pr out for that kind of stuff ;) > > There are no stellar functions in parsers, except what is brought in from > common, w

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Otto Fowler
too bad there isn’t a pr out for that kind of stuff ;) There are no stellar functions in parsers, except what is brought in from common, why doesn’t this just call common? You don’t need parsers jar for any of this do you? On April 24, 2017 at 09:22:06, Casey Stella (ceste...@gmail.com) wrote:

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112951205 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

metron UI

2017-04-24 Thread moshe jarusalem
Hi All, I have recently run metron UI. I am a bit surprised because it has very few features such as configuring some ingestions and topologies. Have I not configured it properly or its features are limited at this time ? Regards,

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 @justinleet - works for me, thanks. --- 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 fea

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112950226 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @mmiklavc @dlyle65535 Do we want to compromise on this for now by removing the dependency, and kick out another ticket to investigate a more robust way to manage this? Mike, you

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/incubator-metron/pull/544 METRON-866 Quickdev and Fulldev readme misses Docker as prerequisite Updates to the README instructions for both "Quick Dev" and "Full Dev". This adds "Docker" as a prerequisite, provi

No longer incubating, but newly hatched!

2017-04-24 Thread Casey Stella
Hi All, Some of you know this already and some of you might not, but as of the last ASF board meeting we became a top level project with me serving as the Vice President of Apache Metron. The good people at the ASF press office scheduled some press early this morning. - NASDAQ GlobeNewswire http

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Casey Stella
Because stellar statements are validated prior to pushing, we needed the functions on the classpath. I don't particularly like it as it's a kludge. I'd really prefer to separate the non-core stellar functions from the Metron core and have them deployed to HDFS, but we haven't gotten there quite y

[GitHub] incubator-metron issue #521: METRON-835 Use Profiler with Kerberos

2017-04-24 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/521 Bump --- 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

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Otto Fowler
So, this script is in common, but: export JAR=metron-parsers-$METRON_VERSION-uber.jar Is the jar it executes. The class it references is : export CLASSNAME=“org.apache.metron.common.cli.ConfigurationManager" So, why is it calling the parsers jar?

Silly question about zk_load_configs.sh

2017-04-24 Thread Otto Fowler
So, this script is in common, but: export JAR=metron-parsers-$METRON_VERSION-uber.jar Is the jar it executes. The class it references is :

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/220 Also - above what you pasted is the real error in the log. that is what would be needed. NPM install is run in a project directory to install any specific node modules that the proj

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread gspeter
Github user gspeter commented on the issue: https://github.com/apache/incubator-metron/pull/220 @simonellistonball ok. Thank you. --- 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 issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/incubator-metron/pull/220 @gspeter would you mind opening this separately as an issue, or a jira (to avoid confusion around this pull request) and posting the output of the -X switch maven suggests. ---

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread gspeter
Github user gspeter commented on the issue: https://github.com/apache/incubator-metron/pull/220 hi @simonellistonball I installed nodejs on Ubuntu Server. ``` curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash - sudo apt-get install -y nodejs ``` --- If

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/incubator-metron/pull/220 @gspeter This looks like an entirely unrelated error. The most likely issue here is that you are missing nodejs from your platform. --- If your project is set up for it, you can

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread gspeter
Github user gspeter commented on the issue: https://github.com/apache/incubator-metron/pull/220 Hi everyone., I have configured according to your instructions. But I got the error below. ``` [INFO] metron-config .. FAILURE [04:14 min