[GitHub] metron issue #599: METRON-975: Normalize logging and switch to common idiom ...

2017-07-17 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/599 @merrimanr Conflicts resolved --- 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 w

[GitHub] metron pull request #620: Metron-988: UI for viewing alerts generated by Met...

2017-07-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/620#discussion_r127832531 --- Diff: metron-interface/metron-alerts/src/app/service/data-source.ts --- @@ -0,0 +1,62 @@ +/** + * Licensed to the Apache Software Foundation (AS

[GitHub] metron pull request #647: METRON-1031: Management UI Cannot Start Topologies...

2017-07-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/647 --- 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 enab

[GitHub] metron issue #647: METRON-1031: Management UI Cannot Start Topologies in Ker...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/647 +1 validated with instructions in the PR description. Nice work! --- 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 doe

[GitHub] metron issue #599: METRON-975: Normalize logging and switch to common idiom ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/599 @mmiklavc looks like there are some merge conflicts. If you don't mind resolving those I will get this reviewed. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] metron issue #599: METRON-975: Normalize logging and switch to common idiom ...

2017-07-17 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/599 I would love to get this in --- 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 wis

[GitHub] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/645 Their status will "Active" but you'll see Kerberos errors in the Storm UI. --- 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] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/645 Ok. So they will "look like" they are running, but they are not actually consuming data. I just wanted to be sure. I will run this through a test so we can get this. --- If your projec

[GitHub] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/645 Without it, the enrichment and indexing topologies don't work on a Kerberized environment because the Storm worker childopts property needs the "-Djava.security.auth.login.config=/path/to/client_ja

[GitHub] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/645 What happens without this PR? What behavior are we correcting? --- 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

[GitHub] metron issue #647: METRON-1031: Management UI Cannot Start Topologies in Ker...

2017-07-17 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/647 @merrimanr Ran it through a full test as described in the PR description and its looking good. Please take a look when you can. --- If your project is set up for it, you can reply to this email

[GitHub] metron issue #359: METRON-569: Change acking to prevent duplicate tuples in ...

2017-07-17 Thread DomenicPuzio
Github user DomenicPuzio commented on the issue: https://github.com/apache/metron/pull/359 Yep, we can wrap it up! Thanks, all! --- 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 en

[GitHub] metron pull request #359: METRON-569: Change acking to prevent duplicate tup...

2017-07-17 Thread DomenicPuzio
Github user DomenicPuzio closed the pull request at: https://github.com/apache/metron/pull/359 --- 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 i

[GitHub] metron issue #359: METRON-569: Change acking to prevent duplicate tuples in ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/359 @DomenicPuzio, this has been resolved by https://github.com/apache/metron/pull/603. Can we close this PR? --- If your project is set up for it, you can reply to this email and have your reply app

Re: Elasticsearch 5.x upgrade

2017-07-17 Thread Otto Fowler
The plan around the mpack with regards to elastic search and solr support and options is extremely important and as of yet undecided. On July 17, 2017 at 03:00:42, Simon Elliston Ball ( si...@simonellistonball.com) wrote: > I assume you're taking about METRON-939. The upgrade is not a huge chang

Re: Elasticsearch 5.x upgrade

2017-07-17 Thread Simon Elliston Ball
I assume you're taking about METRON-939. The upgrade is not a huge change, and unlikely to touch any of the bulk writing pieces. The main thing outstanding on the PR is log4j dependency conflicts from the new ES client. There's no work in that PR to do anything around mpack based install either