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

2017-07-06 Thread iraghumitra
Github user iraghumitra commented on the issue: https://github.com/apache/metron/pull/620 @mraliagha - Do you see a play icon on the UI or a pause icon also can you share the refresh interval that is set. You can click on the sliders icon before paly/pause button to see this

[GitHub] metron issue #619: METRON-939 Elasticsearch ES5 with Xshield client support

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/619 Do we not have to re-write the ambari service? --- 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 #637: METRON-1021 increment metron version number to 0.4.1 in p...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/637 The site/current-book stuff should still refer to current project as 0.4.0? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

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

2017-07-06 Thread mraliagha
Github user mraliagha commented on the issue: https://github.com/apache/metron/pull/620 @iraghumitra I cannot see any error in JS console, except the following warning which I don't think is really important. Angular is running in the development mode. Call enableProdMode() to enab

[GitHub] metron pull request #637: METRON-1021 increment metron version number to 0.4...

2017-07-06 Thread mattf-horton
GitHub user mattf-horton opened a pull request: https://github.com/apache/metron/pull/637 METRON-1021 increment metron version number to 0.4.1 in poms etc ## Contributor Comments Consistent with proposed post-release actions, incrementing the minor release number to 0.4.1, so th

[GitHub] metron pull request #636: METRON-1022: Elasticsearch REST endpoint

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/636#discussion_r126018829 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriter.java --- @@ -60,96 +61,8 @@ pu

[GitHub] metron pull request #636: METRON-1022: Elasticsearch REST endpoint

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/636#discussion_r126018390 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/ElasticsearchServiceImpl.java --- @@ -0,0 +1,74 @@ +/**

[GitHub] metron issue #636: METRON-1022: Elasticsearch REST endpoint

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/636 @merrimanr, I'm obviously feeling a little confused about what seems like 3 related things going on. I don't mean to say this is the wrong approach. Would you feel comfortable opening a [DISC

[GitHub] metron issue #636: METRON-1022: Elasticsearch REST endpoint

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/636 I feel like this and the alert UI are building the transcontinental railroad from each end of the county, and are going to miss each other by a state. How does this relate to that effo

[GitHub] metron pull request #636: METRON-1022: Elasticsearch REST endpoint

2017-07-06 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/636 METRON-1022: Elasticsearch REST endpoint ## Contributor Comments The PR adds a simple Elasticsearch REST endpoint that can execute simple queries. This will serve as the foundation for buildi

[GitHub] metron issue #621: METRON-1001: Allow metron to ingest parser metadata along...

2017-07-06 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/621 Merging master in to get the unit tests to run. --- 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

Re: [VOTE][PROPOSAL] minor changes to release process

2017-07-06 Thread Matt Foley
Thanks, all. That’s 3 binding +1’s, so I’m going to proceed with METRON-1021. Vote needs to stay open 72 hours tho, so if anyone else wishes to vote pro or con, you’ll be listened to. Thanks, --Matt On 7/6/17, 10:24 AM, "Nick Allen" wrote: +1 I think that makes a lot of sense. O

Re: [VOTE][PROPOSAL] minor changes to release process

2017-07-06 Thread Nick Allen
+1 I think that makes a lot of sense. On Wed, Jul 5, 2017 at 5:47 PM, Matt Foley wrote: > (The below proposal is also stated in https://issues.apache.org/ > jira/browse/METRON-1020 ) > > The following proposed changes are small, but not just editorial in > nature, hence will require vote of the

[GitHub] metron issue #631: METRON-1019: Metron 0.4.0 manual installation guide for C...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/631 +1. I believe this is positioned with the correct expectations, and content wise has been worked out on the list ( where this contribution was indeed solicited ). Would like to hear f

[GitHub] metron issue #631: METRON-1019: Metron 0.4.0 manual installation guide for C...

2017-07-06 Thread lvets
Github user lvets commented on the issue: https://github.com/apache/metron/pull/631 Changed the wording in the README.md file to manage expectations. Added month + year. There's a version number in the raw source of "Manual_Install_CentOS6.md". --- If your project is set up f

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/635#discussion_r125928629 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java ---

[GitHub] metron pull request #631: METRON-1019: Metron 0.4.0 manual installation guid...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/631#discussion_r125919953 --- Diff: metron-deployment/other-examples/README.md --- @@ -0,0 +1,3 @@ +# Other Example Deployments --- End diff -- What I would want

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/635#discussion_r125916203 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java ---

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/635#discussion_r125913966 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java --- @@

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/635#discussion_r125910236 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java --- @@

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/635#discussion_r125906152 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java ---

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/635#discussion_r125905841 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchIndexingIntegrationTest.java ---

[GitHub] metron issue #481: METRON-322 Global Batching and Flushing

2017-07-06 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/481 @mattf-horton - Can I offer some help getting back to this? Is a pretty important fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] metron pull request #635: METRON-1018: Integration tests should reference fl...

2017-07-06 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/635 METRON-1018: Integration tests should reference flux yaml and property files deployed by Ambari ## Contributor Comments This PR makes a couple of improvements to our integration tests.

UI pivotting / aggregation backend

2017-07-06 Thread Justin Leet
I wanted to bring up a some stuff on the backend of our UI, and get thoughts (+ things I overlooked, etc.). There's also a couple points at the end that merit discussion about how we handle things, since it gets into how we handle our ES templates (since we generally want to aggregate on raw field

[GitHub] metron pull request #634: METRON-1017: Ambari components should be separate

2017-07-06 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/634 METRON-1017: Ambari components should be separate ## Contributor Comments This PR separates the various configurations in Ambari into different *-env.xml files. Doing this allows us to create

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

2017-07-06 Thread iraghumitra
Github user iraghumitra commented on the issue: https://github.com/apache/metron/pull/620 @mraliagha Can you send me the screenshots for the below two issues and also are you seeing any errors in js console ? - The play/pause button doesn't work properly and creates and infinite

Metron STIG hardening

2017-07-06 Thread Dima Kovalyov
Hello, Has anyone worked on the Metron security side? Maybe it is time to come up with sets of recommendation for hardening Metron installation. We as team are working on applying STIG framework by OpenSCAP, wonder if anyone done anything similar. Thank you. - Dima