[GitHub] incubator-metron pull request: Metron 142 Simplify Parser configur...

2016-05-13 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/120#discussion_r63255049 --- Diff: metron-deployment/roles/metron_streaming/tasks/metron_topology.yml --- @@ -16,7 +16,7 @@ # --- - name: Submit Metron

[GitHub] incubator-metron pull request: Added vagrant install hostmanager s...

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/121 --- 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

[GitHub] incubator-metron pull request: METRON-132 Monit Integration

2016-05-13 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/118#issuecomment-219158724 +1, looks great! --- 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 pull request: [METRON-150] Adding WebSphere parse...

2016-05-13 Thread DomenicPuzio
Github user DomenicPuzio commented on the pull request: https://github.com/apache/incubator-metron/pull/115#issuecomment-219095558 @merrimanr and @cestella, thank you SO MUCH for the help! I really appreciate it! I will certainly follow the steps to test on vagrant for future

[GitHub] incubator-metron pull request: Added vagrant install hostmanager s...

2016-05-13 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/121#issuecomment-219068764 +1 --- 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

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-13 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/119#issuecomment-219059773 I think that makes sense, especially since this creates the 'hook' for later contributions of BPF. The only downside is that we don't want to

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-13 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/119#issuecomment-219049457 @nickwallen Agreed we should support BPF. This PR makes the filter pluggable and we already have the query language. We can have a follow-on PR for BPF

[GitHub] incubator-metron pull request: [METRON-150] Adding WebSphere parse...

2016-05-13 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/115#issuecomment-219048736 Ok, tested this. Sorry it took so long (vagrant drama). Ok, so this is what I did to test this. Spin up the full-dev-vagrant: 1. Do a build