[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 by

[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 supp

[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-219048690 For some reason, I cannot comment on the original JIRA, so I will comment here. I think using a custom created language for this purpose is going to

[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-219048777 +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-150] Adding WebSphere parse...

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

[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 confus

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

2016-05-13 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/incubator-metron/pull/121 Added vagrant install hostmanager step Added missing step to the README You can merge this pull request into a Git repository by running: $ git pull https://github.com/merrimanr/inc

[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-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 parsers.

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

2016-05-13 Thread dlyle65535
Github user dlyle65535 commented on the pull request: https://github.com/apache/incubator-metron/pull/121#issuecomment-219140923 +1 - ship it! :) --- 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 d

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

2016-05-13 Thread dlyle65535
Github user dlyle65535 commented on the pull request: https://github.com/apache/incubator-metron/pull/118#issuecomment-219145861 +1 looks great, can't wait to use it! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[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 have

[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 featu

[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 Par