[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-27 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/441 +1 from me --- 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

[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-22 Thread kylerichardson
Github user kylerichardson commented on the issue: https://github.com/apache/incubator-metron/pull/441 Thanks, @merrimanr! I'll get that change pushed out later today. Waiting on #462 to be merged to fix the build errors we're seeing and then I'll rebase and push. On

[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-22 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/441 Works great. I have just one very small request. Can you update the Usage section of the README (where it lists the UI addresses) to point to the Elasticsearch head plugin instead of

[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-21 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/441 @kylerichardson I will get this reviewed 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

[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-21 Thread kylerichardson
Github user kylerichardson commented on the issue: https://github.com/apache/incubator-metron/pull/441 bump... I read back through the discuss thread from the dev list and we didn't land on any specifics. There seems to be general agreement that metron-docker: - fulfills a

[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-05 Thread kylerichardson
Github user kylerichardson commented on the issue: https://github.com/apache/incubator-metron/pull/441 Good point, @nickwallen. My hope for metron-docker is as a lightweight alternative to vagrant for development. That said, I would hate to see it become a large overhead for the

[GitHub] incubator-metron issue #441: METRON-646: Add index templates to metron-docke...

2017-02-05 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/441 Hi @kylerichardson - I don't want to throw cold water on your effort, but I am hesitant to create a third deployment code base for `metron-docker` (in addition to MPack and Ansible.)