[GitHub] metron issue #797: METRON-1243: Add a REST endpoint which allows us to get a...

2017-10-25 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/797 and by downstream, I mean in whatever UI we have. ---

[GitHub] metron issue #797: METRON-1243: Add a REST endpoint which allows us to get a...

2017-10-25 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/797 So, I did think about this and decided to include it. It is an index in ES with data. If we think it should be filtered, I can certainly do so, but thought the decision really should be best done

[GitHub] metron issue #797: METRON-1243: Add a REST endpoint which allows us to get a...

2017-10-25 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/797 Just tested this and worked as expected. The only question I have is whether the "error" index should be included since it is used to store errors and not alerts. I can't think of a case where

[GitHub] metron issue #797: METRON-1243: Add a REST endpoint which allows us to get a...

2017-10-11 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/797 # Testing Plan ## Basic Functionality * Ensure you can retrieve the elasticsearch indices set up in Metron and they include the default sensors: `curl -u user:password -X GET

[GitHub] metron issue #797: METRON-1243: Add a REST endpoint which allows us to get a...

2017-10-10 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/797 Nevermind, I was mobile before, took care of it ---

[GitHub] metron issue #797: METRON-1243: Add a REST endpoint which allows us to get a...

2017-10-10 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/797 Can you assign METRON-1216 to you and link / complete it as necessary? Thanks! ---