[GitHub] incubator-metron pull request #502: METRON-814 minor tweaks in document form...

GitHub user mattf-horton opened a pull request: https://github.com/apache/incubator-metron/pull/502 METRON-814 minor tweaks in document format of Kerberos-setup.md ## Contributor Comments Apparently github ate my latest comment on PR#497. Here's the fix that would have been

[GitHub] incubator-metron pull request #497: METRON-804: Create a document to describ...

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

Re: [DISCUSS] The bro kafka plugin

I would be happy to try it again but I attempted to do that before with bro packages and it failed to be able to handle it. I also tried using branches of a repo with bro but that similarly failed (and was a pretty bad idea to start with). Jon On Fri, Mar 31, 2017, 3:24 PM Matt Foley

Re: [DISCUSS] The bro kafka plugin

We should be able to request just one alternate repo from INFRA, and put a top hierarchical level in it that doesn’t include a maven pom. As far as maven and clients are concerned, it just increases by 1 the path length to the root of the repo. On 3/31/17, 10:30 AM, "zeo...@gmail.com"

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/497 +1 by inspection and via vagrant. --- 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: configuration update during runtime

Hi Moshe, Dynamic configuration, with parameters stored in Zookeeper, is actually quite complex if you follow it from end to end. Here’s a brief description: • The Metron configuration parameter mechanism uses Apache Curator to subscribe to ZK nodes of interest, and maintain a TreeCache with

[GitHub] incubator-metron issue #499: METRON-196: Deployment Fails Without Ansible 2....

Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/499 @JonZeolla - That looks like data was sent before the templates were loaded. Is that possible? I had clean data on all my test runs. --- If your project is set up for it, you can reply

[GitHub] incubator-metron issue #499: METRON-196: Deployment Fails Without Ansible 2....

Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/499 I'm getting `{"dns": {"ts":1490980024.00.58808,"uid":"CmLZ8q1IzktXz75Bq"` from sensor-stubs, where it should be `{"dns": {"ts":1484169493.58808,"uid":"CmLZ8q1IzktXz75Bq"` according

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109211360 --- Diff: metron-interface/metron-config/e2e/sensor-config-readonly/sensor-config-readonly.po.ts --- @@ -0,0 +1,125 @@ +/** + *

Re: [DISCUSS] The bro kafka plugin

Once we agree on a repo location to host this, I would be happy to put together the package and update our environments to use bro-pkg to install the plugin. I have created METRON-813 to track this and changed METRON-348

[GitHub] incubator-metron issue #219: METRON-196 Fix support of ansible 2.1 in metron...

Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/219 This is obsolete as of #499. --- 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 pull request #499: METRON-196: Deployment Fails Without Ans...

Github user JonZeolla commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/499#discussion_r109187061 --- Diff: metron-deployment/amazon-ec2/README.md --- @@ -12,11 +12,11 @@ Getting Started The host used to deploy Apache Metron will

[GitHub] incubator-metron pull request #499: METRON-196: Deployment Fails Without Ans...

Github user JonZeolla commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/499#discussion_r109186296 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -30,25 +30,19 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109204340 --- Diff: metron-interface/metron-config/e2e/sensor-config-readonly/sensor-config-readonly.po.ts --- @@ -0,0 +1,125 @@ +/** + *

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109203096 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/GrokController.java --- @@ -53,4 +54,11 @@

Re: [DISCUSS] The bro kafka plugin

Could we create a separate repo for more than on thing? like put … um let’s say a maven plugin and the bro plugin? On March 31, 2017 at 12:30:25, Nick Allen (n...@nickallen.org) wrote: I agree with everything that I've read. One of the guys from Bro had contacted me a while back, letting me

[GitHub] incubator-metron pull request #497: METRON-804: Create a document to describ...

Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/497#discussion_r109200716 --- Diff: metron-deployment/vagrant/KERBEROS_SETUP.md --- @@ -0,0 +1,238 @@ +# Setting Up Kerberos in Vagrant Full Dev +**Note:** These

Re: [DISCUSS] The bro kafka plugin

I agree with everything that I've read. One of the guys from Bro had contacted me a while back, letting me know that the packaging mechanism in Bro was ready for public consumption. I just have not had cycles to do anything with it yet. They are not wanting to host any of the plugins. I

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109198964 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.ts --- @@ -0,0 +1,82 @@ +/** + *

[GitHub] incubator-metron pull request #411: METRON-635: Vagrant provisioning fails o...

Github user JonZeolla closed the pull request at: https://github.com/apache/incubator-metron/pull/411 --- 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] incubator-metron issue #411: METRON-635: Vagrant provisioning fails on CentO...

Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/411 Vagrant provisioning [isn't supported on CentOS 6.8](https://github.com/apache/incubator-metron/pull/436/commits/37ddde3ae2f26e4efebe1c1c3d2e43a4c70f4b0b) any more, so removing this PR.

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109198348 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.ts --- @@ -0,0 +1,82 @@ +/** + * Licensed

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109196352 --- Diff: metron-interface/metron-config/e2e/sensor-config-readonly/sensor-config-readonly.po.ts --- @@ -0,0 +1,125 @@ +/** + *

[GitHub] incubator-metron issue #489: METRON-623: Management UI

Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/489 @simonellistonball I totally agree. I would rather have this in to work off of. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109191525 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/GrokController.java --- @@ -53,4 +54,11 @@

[GitHub] incubator-metron pull request #497: METRON-804: Create a document to describ...

Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/497#discussion_r109190138 --- Diff: metron-deployment/vagrant/KERBEROS_SETUP.md --- @@ -0,0 +1,238 @@ +# Setting Up Kerberos in Vagrant Full Dev +**Note:** These

[GitHub] incubator-metron issue #489: METRON-623: Management UI

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/489 @simonellistonball I'm ok with iterating and accepting this PR. I would ask that on the pending architectural document that is owed by the top level comment that we address it. --- If

[GitHub] incubator-metron issue #489: METRON-623: Management UI

Github user simonellistonball commented on the issue: https://github.com/apache/incubator-metron/pull/489 I completely agree that right now this is too grok specific, but I would suggest we try and get this PR in, and then handle changes to generalise. That way we can account for

[GitHub] incubator-metron issue #489: METRON-623: Management UI

Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/489 I agree with @cestella about making this grok centric. We should have a discussion around making the rest service work with plugins ( for example each

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109186947 --- Diff: metron-interface/metron-config/src/app/model/threat-triage-config.ts --- @@ -0,0 +1,23 @@ +import {RiskLevelRule} from

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109186782 --- Diff: metron-interface/metron-config/src/app/model/threat-triage-config.ts --- @@ -0,0 +1,23 @@ +import {RiskLevelRule} from

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109186326 --- Diff: metron-interface/metron-config/src/app/model/threat-triage-config.ts --- @@ -0,0 +1,23 @@ +import {RiskLevelRule} from

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109185935 --- Diff: metron-interface/metron-config/src/app/model/parse-message-request.ts --- @@ -0,0 +1,23 @@ +/** + * Licensed to the

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109185069 --- Diff: metron-interface/metron-config/src/app/model/threat-triage-config.ts --- @@ -0,0 +1,23 @@ +import {RiskLevelRule} from

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109185011 --- Diff: metron-interface/metron-config/src/app/model/parse-message-request.ts --- @@ -0,0 +1,23 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #501: METRON-812: Make the bro-kafka plugin wo...

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/501#discussion_r109178739 --- Diff: metron-sensors/bro-plugin-kafka/README.md --- @@ -0,0 +1,160 @@ +Bro Logging Output to Kafka +=== +

[GitHub] incubator-metron pull request #501: METRON-812: Make the bro-kafka plugin wo...

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/501#discussion_r109178644 --- Diff: metron-sensors/bro-plugin-kafka/README.md --- @@ -0,0 +1,160 @@ +Bro Logging Output to Kafka +=== +

[GitHub] incubator-metron pull request #501: METRON-812: Make the bro-kafka plugin wo...

Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/501#discussion_r109178396 --- Diff: metron-sensors/bro-plugin-kafka/README.md --- @@ -0,0 +1,160 @@ +Bro Logging Output to Kafka +===

[GitHub] incubator-metron pull request #501: METRON-812: Make the bro-kafka plugin wo...

Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/501#discussion_r109171010 --- Diff: metron-sensors/bro-plugin-kafka/README.md --- @@ -0,0 +1,160 @@ +Bro Logging Output to Kafka +===

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r108071204 --- Diff: metron-interface/metron-config/src/styles.scss --- @@ -0,0 +1,739 @@ +/** + * Licensed to the Apache Software

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109168862 --- Diff: metron-interface/metron-config/src/app/login/login.component.html --- @@ -0,0 +1,31 @@ + + + --- End diff --

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109169645 --- Diff: metron-interface/metron-config/angular-cli.json --- @@ -0,0 +1,51 @@ +{ + "project": { +"version":

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109164883 --- Diff: metron-interface/metron-config/README.md --- @@ -0,0 +1,67 @@ +# Metron Management UI + +This module provides a user

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166424 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166288 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109168190 --- Diff: pom.xml --- @@ -274,7 +274,13 @@ **/hbase/data/** **/kafkazk/data/**

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109164596 --- Diff: metron-interface/metron-config/README.md --- @@ -0,0 +1,67 @@ +# Metron Management UI + +This module provides a user

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109167480 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/GrokController.java --- @@ -53,4 +54,11 @@

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109167063 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-parser-config-readonly/sensor-parser-config-readonly.component.html --- @@ -0,0

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109168022 --- Diff: pom.xml --- @@ -274,7 +274,13 @@ **/hbase/data/** **/kafkazk/data/**

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166373 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166325 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166178 --- Diff: metron-interface/metron-config/e2e/sensor-config-readonly/sensor-config-readonly.po.ts --- @@ -0,0 +1,125 @@ +/** + * Licensed

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166443 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109165913 --- Diff: metron-interface/metron-config/src/app/model/parse-message-request.ts --- @@ -0,0 +1,23 @@ +/** + * Licensed to the Apache

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user iraghumitra commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109168211 --- Diff: metron-interface/metron-config/src/app/login/login.component.html --- @@ -0,0 +1,31 @@ + + + --- End diff --

[GitHub] incubator-metron pull request #489: METRON-623: Management UI

Github user iraghumitra commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r109166246 --- Diff: metron-interface/metron-config/angular-cli.json --- @@ -0,0 +1,51 @@ +{ + "project": { +"version": "1.0.0-beta.15",

[GitHub] incubator-metron issue #499: METRON-196: Deployment Fails Without Ansible 2....

Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/499 Taking a look at this today. --- 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 #499: METRON-196: Deployment Fails Without Ansible 2....

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/499 This is great work @dlyle65535 I'll reserve my +1 for immediately after Jon tries it out. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-metron pull request #500: METRON-795: Install Metron REST with Amb...

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/500#discussion_r109159403 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/templates/application.yml.j2

configuration update during runtime

Hi All, I have been looking the codes for ConfiguredBolt and its derivatives. I realized that updateConfig is actually not doing much? Would you describe how you manage configuration changes might be needed after bolts are initialized and running? for convenience, I copied the code here public

[GitHub] incubator-metron pull request #488: METRON-796: Mpack uses wrong group for o...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/488 --- 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] incubator-metron pull request #500: METRON-795: Install Metron REST with Amb...

Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/500#discussion_r109141233 --- Diff:

[GitHub] incubator-metron issue #501: METRON-812: Make the bro-kafka plugin work with...

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