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

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/530 Running the generate command in CentOS results in defaulting to plugin version 3.0.1 when run from a directory without a pom. In Mac, I'm getting 2.4 for some reason. I haven't found any settings

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

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/620 These lic. check commands needs to make their way into the build, like we do with the java stuff. --- If your project is set up for it, you can reply to this email and have your reply appear

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

Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/620 If we just focus on the production dependencies, which AFAIK is all we need to worry about, then I think the licenses are not a problem. ``` $ license-checker --production | grep

[GitHub] metron pull request #626: METRON-947 [DO NOT MERGE UNTIL METRON-942] Stellar...

Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/626 --- 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 #626: METRON-947 [DO NOT MERGE UNTIL METRON-942] Stellar Manage...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/626 I am closing this PR, I'll re-submit when METRON-942 is landed --- 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 issue #530: METRON-777 Metron Extension System and Parser Extensions

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/530 I just noticed that you had addressed my findings around the plugin version as well. So I think going back to 3.0.1 is ok for now. We do need to figure out what is going on with Mac OS before

[GitHub] metron pull request #666: METRON-1051: Enable the ability to update indexed ...

Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/666 --- 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

[GitHub] metron issue #687: METRON-1090 [NO MERGE UNTIL METRON-711] Add Assignment to...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/687 Added pre and post increment and decrement --- 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

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

Github user justinleet commented on the issue: https://github.com/apache/metron/pull/673 Closed via manually rebased commits to maintain attribution. Commits are: 3d4751 c7d8b7 ed689c --- If your project is set up for it, you can reply to this email and have your

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

Github user justinleet closed the pull request at: https://github.com/apache/metron/pull/673 --- 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

[GitHub] metron issue #683: METRON-1084: Management UI web server license should be A...

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/683 @justinleet there are actually 2 package.json files in the management UI module. I just fixed the other one too. Can you check it out again? --- If your project is set up for it, you can reply

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

Github user justinleet commented on the issue: https://github.com/apache/metron/pull/673 @cestella Is there anything else you'd like to see here now that a couple more commits were added? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] metron issue #528: METRON-838 Incorrect set of ts in FireEye parser

Github user bjigmp commented on the issue: https://github.com/apache/metron/pull/528 Thanks @justinleet. Changed to ZoneOffset.UTC --- 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

[GitHub] metron issue #683: METRON-1084: Management UI web server license should be A...

Github user justinleet commented on the issue: https://github.com/apache/metron/pull/683 +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 the

[GitHub] metron pull request #683: METRON-1084: Management UI web server license shou...

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/683#discussion_r131652946 --- Diff: metron-interface/metron-config/scripts/package.json --- @@ -18,5 +18,5 @@ }, "private": true, "author": "", -

[GitHub] metron pull request #683: METRON-1084: Management UI web server license shou...

Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/683#discussion_r131651567 --- Diff: metron-interface/metron-config/scripts/package.json --- @@ -18,5 +18,5 @@ }, "private": true, "author": "", -

Re: Upgrade vagrant base to centos 7

Ahh, perfect, I didn't know we stored that in the repo too. I will take a look thanks. Jon On Mon, Aug 7, 2017 at 8:18 AM David Lyle wrote: > Hi Jon, > > I'm not sure how to answer your question, but the base image (used for > full-dev) we use is defined here: > >

[GitHub] metron issue #680: METRON-1078: Metron Indexing fails to stop during "Stop A...

Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/680 @justinleet - this seems better. The change is to not error when the rest service is unavailable but use that as an indication that the topologies are stopped. The rest is code formatting because

Re: Upgrade vagrant base to centos 7

Hi Jon, I'm not sure how to answer your question, but the base image (used for full-dev) we use is defined here: https://github.com/apache/metron/blob/master/metron-deployment/packaging/packer-build/base-centos-6.7.json. It's forked from bento/centos6.7, but not derived from it. Same deal with

[GitHub] metron issue #528: METRON-838 Incorrect set of ts in FireEye parser

Github user justinleet commented on the issue: https://github.com/apache/metron/pull/528 +1 to this. There's a review comment, but quite frankly I consider it optional. If @bjigmp doesn't want to make that change, I'm still good. --- If your project is set up for it, you can reply

[GitHub] metron pull request #528: METRON-838 Incorrect set of ts in FireEye parser

Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/528#discussion_r131635959 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/fireeye/BasicFireEyeParserTest.java --- @@ -57,4 +62,16 @@ public void