Re: Metron Installation on an Ambari-Managed Cluster?

2017-03-24 Thread Otto Fowler
I am looking for a fully scriptable solution, where I don’t have to go into ambari, such as running the ansible playbook. Before I would run build and playbook from docker. On March 24, 2017 at 19:34:12, Matt Foley (ma...@apache.org) wrote: https://cwiki.apache.org/confluence/display/METRON/Metr

[DISCUSS] Kerberos Support

2017-03-24 Thread Casey Stella
Hi All, I'd like to talk and start to formulate a plan around supporting running Metron on a kerberized cluster. This is a big bundle of work and seems dauntingly nebulous, so I wanted to have a chat and get a firm direction. When initially contemplating the issue, it's apparent that there are a

[GitHub] incubator-metron issue #486: METRON-793: Migrate to storm-kafka-client kafka...

2017-03-24 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/486 Ok, update on the integration tests. It appears that the issue is that storm 1.0.3 does not consistently kill the slots when shutting down. It times out after a minute and dies. The fix

Re: Metron Installation on an Ambari-Managed Cluster?

2017-03-24 Thread Matt Foley
https://cwiki.apache.org/confluence/display/METRON/Metron+with+HDP+2.5+bare-metal+install is much more recent. It should still work, but I’m not sure how RPM generation has changed, especially after David’s changes in METRON-671 (Refactor Ansible deployment to use Ambari). Of course the two sce

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107988742 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.scss --- @@ -0,0 +1,168 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107987263 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.html --- @@ -0,0 +1,113 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r108007221 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.spec.ts --- @@ -0,0 +1,506 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107987434 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.html --- @@ -0,0 +1,113 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107953775 --- 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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107987338 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.html --- @@ -0,0 +1,113 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107956884 --- Diff: metron-interface/metron-config/src/app/login/login.component.spec.ts --- @@ -0,0 +1,65 @@ +/** + * Licensed to the Apac

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r108012411 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.spec.ts --- @@ -0,0 +1,506 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107988671 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.scss --- @@ -0,0 +1,168 @@

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

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

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

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

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107954344 --- Diff: metron-deployment/packaging/docker/rpm-docker/SPECS/metron.spec --- @@ -313,6 +315,25 @@ This package installs the Metron Profi

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107988841 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-field-schema/sensor-field-schema.component.scss --- @@ -0,0 +1,168 @@

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107977639 --- Diff: metron-interface/metron-config/src/app/model/topology-status.ts --- @@ -0,0 +1,26 @@ +/** + * Licensed to the Apache So

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

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

Metron Installation on an Ambari-Managed Cluster?

2017-03-24 Thread Otto Fowler
I have used https://cwiki.apache.org/confluence/display/METRON/Metron+Installation+on+an+Ambari-Managed+Cluster as a guide in the past, but it is out of date, I don’t think it can possibly work with docker rpm build now. Does anyone have any ideas what it would take to get this workflow working ag

[GitHub] incubator-metron issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/incubator-metron/pull/488 +1 then, thanks. Please expand METRON-349 to include these issues too. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-metron issue #490: DO NOT MERGE METRON-797: Pass security.protocol...

2017-03-24 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/incubator-metron/pull/490 Thanks, there didn't seem to be a jira for parsers, so I expanded the scope of METRON-798 and made bro it's first sub-task. --- If your project is set up for it, you can reply to this

[GitHub] incubator-metron issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/incubator-metron/pull/488 I agree, let's get this sorted, and then follow up with a general review of the permissions once we've got the rest of the security pieces in place. --- If your project is set up

[GitHub] incubator-metron issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/488 @mattf-horton Permissions on the other items are generally 775, so they can be read as needed (and should be scaled back once we have everything lined up with the user as @simonellistonb

[GitHub] incubator-metron issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/incubator-metron/pull/488 Thanks. Looks okay to me now, for the intended purpose. +0 due to Simon's concerns; I'd prefer someone with more experience in Metron give you the plus one. Regarding @simon

[GitHub] incubator-metron issue #490: DO NOT MERGE METRON-797: Pass security.protocol...

2017-03-24 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/490 Good catch. I made the dependency explicit and created the JIRAs that I know about for kerberos [here](https://issues.apache.org/jira/browse/METRON-802?jql=labels%20%3D%20kerberos%20AND%2

[GitHub] incubator-metron issue #486: METRON-793: Migrate to storm-kafka-client kafka...

2017-03-24 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/486 I found a couple of spots where we might not be cleaning things up and exceptions that are thrown that kill the spout that shouldn't. I've run the test 10 times locally and will continue

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

2017-03-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/488#discussion_r107965896 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/params/params_linux.py

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

2017-03-24 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/488#discussion_r107964537 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/params/params_linux.py

[GitHub] incubator-metron issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/488 For anybody looking, Matt's review comment is still relevant to discussion, but unfortunately hidden by GitHub thinking it's outdated after the last commit. --- If your project is set u

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

2017-03-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/488#discussion_r107961358 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/params/params_linux.py

[GitHub] incubator-metron issue #490: DO NOT MERGE METRON-797: Pass security.protocol...

2017-03-24 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/incubator-metron/pull/490 @cestella , In the Jira, please add the dependencies as blockers (at least METRON-793), and create (if it doesn't exist yet) the jira(s) for the needed follow-on work and link to them

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

2017-03-24 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/488#discussion_r107957631 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/params/params_linux.py

[GitHub] incubator-metron issue #486: METRON-793: Migrate to storm-kafka-client kafka...

2017-03-24 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/486 @justinleet It is unclear and that test has had some intermittent issues, especially under load, I've noticed. I added a more descriptive message to help diagnose. --- If your project i

[GitHub] incubator-metron issue #486: METRON-793: Migrate to storm-kafka-client kafka...

2017-03-24 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/486 @justinleet Thanks for checking this out - modified the instructions with the ACL command corrections. Must have copy-pasted the wrong commands from history, so thanks for that! --- If y

[GitHub] incubator-metron issue #486: METRON-793: Migrate to storm-kafka-client kafka...

2017-03-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/486 @cestella Just noticed Travis after I commented. I'm moderately surprised that the most recent PR would break it, do you know what the issue is? --- If your project is set up for it, y

[GitHub] incubator-metron issue #486: METRON-793: Migrate to storm-kafka-client kafka...

2017-03-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/486 +1, was able to follow Mike's instructions, with a couple caveats. - Group authorization command was missing ``` /usr/hdp/current/kafka-broker/bin/kafka-acls.sh --authoriz

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

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

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

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

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

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

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932760 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.html --- @@ -0,0 +1,107 @@ + +

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932746 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.html --- @@ -0,0 +1,107 @@ + +

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932680 --- Diff: metron-interface/metron-config/package.json --- @@ -0,0 +1,65 @@ +{ + "name": "clitest", + "version": "0.0.0",

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932702 --- Diff: metron-interface/metron-config/src/app/_main.scss --- @@ -0,0 +1,113 @@ +/** + * Licensed to the Apache Software Foundat

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

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

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932715 --- Diff: metron-interface/metron-config/src/app/app.component.ts --- @@ -0,0 +1,41 @@ +/** + * Licensed to the Apache Software Fo

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932690 --- Diff: metron-interface/metron-config/scripts/start_management_ui.sh --- @@ -0,0 +1,27 @@ +#!/bin/bash +# +# Licensed to t

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932735 --- Diff: metron-interface/metron-config/src/app/app.component.ts --- @@ -0,0 +1,41 @@ +/** + * Licensed to the Apache Software Fo

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932655 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apa

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932605 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-parser-config-readonly/sensor-parser-config-readonly.component.html ---

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932661 --- Diff: metron-interface/metron-config/e2e/sensor-list/sensor-list.po.ts --- @@ -0,0 +1,240 @@ +/** + * Licensed to the Apache

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932671 --- Diff: metron-interface/metron-config/e2e/sensor-list/sensor-list.po.ts --- @@ -0,0 +1,240 @@ +/** + * Licensed to the Apache

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932638 --- 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

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

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107932616 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-parser-config-readonly/sensor-parser-config-readonly.component.html ---

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/489#discussion_r107931875 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-grok/sensor-grok.component.html --- @@ -0,0 +1,42 @@ + + +

[GitHub] incubator-metron issue #490: DO NOT MERGE METRON-797: Pass security.protocol...

2017-03-24 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/490 # Testing Plan ## Preliminaries * Please perform the following tests on the `full-dev` vagrant environment. * Set an environment variable to indicate `METRON_HOME`:

[GitHub] incubator-metron pull request #490: DO NOT MERGE METRON-797: Pass security.p...

2017-03-24 Thread cestella
Github user cestella closed the pull request at: https://github.com/apache/incubator-metron/pull/490 --- 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 fea

[GitHub] incubator-metron pull request #490: DO NOT MERGE METRON-797: Pass security.p...

2017-03-24 Thread cestella
GitHub user cestella reopened a pull request: https://github.com/apache/incubator-metron/pull/490 DO NOT MERGE METRON-797: Pass security.protocol and enable auto-renew for the storm topologies ## Contributor Comments METRON-793 migrated the storm topologies to the storm-kaf

[GitHub] incubator-metron pull request #486: METRON-793: Migrate to storm-kafka-clien...

2017-03-24 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/486#discussion_r107915490 --- Diff: metron-platform/metron-storm-kafka/src/main/java/org/apache/metron/storm/kafka/flux/SimpleStormKafkaBuilder.java --- @@ -0,0 +1,234 @@

[GitHub] incubator-metron pull request #486: METRON-793: Migrate to storm-kafka-clien...

2017-03-24 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/486#discussion_r107915471 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/topology/ParserTopologyBuilder.java --- @@ -106,19 +105,22 @@ pu

[GitHub] incubator-metron pull request #486: METRON-793: Migrate to storm-kafka-clien...

2017-03-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/486#discussion_r107913632 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/topology/ParserTopologyBuilder.java --- @@ -106,19 +105,22 @@

[GitHub] incubator-metron pull request #486: METRON-793: Migrate to storm-kafka-clien...

2017-03-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/486#discussion_r107912514 --- Diff: metron-platform/metron-storm-kafka/src/main/java/org/apache/metron/storm/kafka/flux/SimpleStormKafkaBuilder.java --- @@ -0,0 +1,234 @

[GitHub] incubator-metron issue #490: DO NOT MERGE METRON-797: Pass security.protocol...

2017-03-24 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/490 If you want to review just this work independent of 793 prior to 793 getting merged, you can find the commits: * [here](https://github.com/apache/incubator-metron/pull/490/commits/5e61

[GitHub] incubator-metron pull request #490: DO NOT MERGE METRON-797: Pass security.p...

2017-03-24 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/incubator-metron/pull/490 DO NOT MERGE METRON-797: Pass security.protocol and enable auto-renew for the storm topologies ## Contributor Comments METRON-793 migrated the storm topologies to the storm-kafka

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107904482 --- Diff: metron-interface/metron-config/src/app/app.component.ts --- @@ -0,0 +1,41 @@ +/** + * Licensed to the Apache Software Fo

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107892970 --- Diff: metron-interface/metron-config/e2e/sensor-config-readonly/sensor-config-readonly.po.ts --- @@ -0,0 +1,125 @@ +/** + *

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107905877 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.spec.ts --- @@ -0,0 +1,161 @@ +/**

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107894517 --- Diff: metron-interface/metron-config/e2e/sensor-list/sensor-list.po.ts --- @@ -0,0 +1,240 @@ +/** + * Licensed to the Apache

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

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

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107719953 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-parser-config-readonly/sensor-parser-config-readonly.component.html ---

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107894592 --- Diff: metron-interface/metron-config/e2e/sensor-list/sensor-list.po.ts --- @@ -0,0 +1,240 @@ +/** + * Licensed to the Apache

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107902092 --- Diff: metron-interface/metron-config/scripts/start_management_ui.sh --- @@ -0,0 +1,27 @@ +#!/bin/bash +# +# Licensed to t

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107719782 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-parser-config-readonly/sensor-parser-config-readonly.component.html ---

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107904759 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.html --- @@ -0,0 +1,107 @@ + +

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107904887 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.html --- @@ -0,0 +1,107 @@ + +

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107900720 --- Diff: metron-interface/metron-config/package.json --- @@ -0,0 +1,65 @@ +{ + "name": "clitest", + "version": "0.0.0",

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107905795 --- Diff: metron-interface/metron-config/src/app/general-settings/general-settings.component.spec.ts --- @@ -0,0 +1,161 @@ +/**

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107904541 --- Diff: metron-interface/metron-config/src/app/app.component.ts --- @@ -0,0 +1,41 @@ +/** + * Licensed to the Apache Software Fo

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

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

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107902487 --- Diff: metron-interface/metron-config/src/app/_main.scss --- @@ -0,0 +1,113 @@ +/** + * Licensed to the Apache Software Foundat

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107893194 --- Diff: metron-interface/metron-config/e2e/sensor-config/sensor-config.po.ts --- @@ -0,0 +1,243 @@ +/** + * Licensed to the Apa

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107892229 --- Diff: metron-interface/metron-config/angular-cli.json --- @@ -0,0 +1,51 @@ +{ + "project": { +"version": "1.0.0-beta.

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

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/484#discussion_r107719261 --- Diff: metron-interface/metron-config/src/app/sensors/sensor-grok/sensor-grok.component.html --- @@ -0,0 +1,42 @@ + + +

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

2017-03-24 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/incubator-metron/pull/489 METRON-623: Management UI ## Contributor Comments This PR includes the latest version of the Management UI. Instructions for building and installing are located in metron-interface

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

2017-03-24 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/incubator-metron/pull/484 --- 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 #484: METRON-623: Management UI

2017-03-24 Thread merrimanr
GitHub user merrimanr reopened a pull request: https://github.com/apache/incubator-metron/pull/484 METRON-623: Management UI ## Contributor Comments This PR includes the latest version of the Management UI. Instructions for building and installing are located in metron-interfa

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

2017-03-24 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/incubator-metron/pull/484 --- 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 issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/488 I agree the topologies should run as the metron user, but this is just to get things back in a working state again (and it already used to be this way, so this isn't opening things up mo

[GitHub] incubator-metron issue #488: METRON-796: Mpack uses wrong group for owning H...

2017-03-24 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/incubator-metron/pull/488 Opening this to the hadoop group feels wrong from a security perspective. That gives all the other hadoop users too much write access to the data store. A better solution would be

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

2017-03-24 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/incubator-metron/pull/488 METRON-796: Mpack uses wrong group for owning HDFS directories ## Contributor Comments Reverts the group owner of a couple HDFS directories to be the hadoop group, rather than the m