[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/774 > @ottobackwards hopefully the Readme changes help. The Getting Started information that you added to the README is outstanding! ---

[GitHub] metron pull request #805: METRON-1261: Apply bro security patch

2017-10-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/805 ---

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/metron/pull/774 @ottobackwards says: > would it be possible for you to help us with a comment as to your approach when you reviewed this functionality? It might help if you could summarize your feelings as

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/774 @nickwallen hopefully the Readme changes help. ---

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/774 I will extend the readme ---

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/774 @nickwallen If is OK with you, i'll make it a separate file to the README.md ---

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/774 Thanks for looking at this again Nick, let me try to answer your questions. TL;DR I'm sorry for my confusion with your original statement. I had in mind this part of

[GitHub] metron issue #805: METRON-1261: Apply bro security patch

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/805 +1 Followed instructions and everything worked as expected. Great work @JonZeolla! ---

[GitHub] metron issue #805: METRON-1261: Apply bro security patch

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/805 never mind, I just re-read the instructions before sorry ---

[GitHub] metron issue #805: METRON-1261: Apply bro security patch

2017-10-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/805 @JonZeolla. Followed the steps, and in my monitor storm logs 'window' i have many of the following: ```bash [root@node1 ~]# tail -f /var/log/storm/workers-artifacts/indexing-*/*/w

[GitHub] metron pull request #818: METRON-1284: Remove extraneous dead query in Elast...

2017-10-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/818 ---

[GitHub] metron issue #818: METRON-1284: Remove extraneous dead query in Elasticsearc...

2017-10-27 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/818 +1 by inspection. ---

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

2017-10-27 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/774 > Test Project can be examined as a simple example of how to create bundles. @ottobackwards Maybe I'm being really stupid, but I don't understand how the test-bundles-plugin shows me

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-27 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/817 > Hi @nickwallen - sure, what you are saying makes sense to me. To address point 1, I have moved the ES template installation step into the Indexing "Install" method, Did this work for yo

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-27 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/metron/pull/817 ---

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-27 Thread anandsubbu
GitHub user anandsubbu reopened a pull request: https://github.com/apache/metron/pull/817 METRON-1283: Install Elasticsearch template as a part of the mpack startup scripts ## Contributor Comments For a Metron multi-node deployment using mpack, the Elasticsearch template i

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-27 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 Restarting travis ---

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-27 Thread anandsubbu
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/817 Hi @ottobackwards , made the change to remove the ES template installation from 'load_web_template' role for vagrant deployments. Fired up full-dev with the change and it comes up fine. Was able t