[GitHub] metron issue #905: METRON-1417: Disable pcap-service by default in Monit

2018-01-24 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/905 Thanks @nickwallen, I plan to spend some time reviewing 903 as well. ---

[GitHub] metron issue #905: METRON-1417: Disable pcap-service by default in Monit

2018-01-24 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/905 I should also say, #903 certainly doesn't preclude this. This has a +1 from me. ---

[GitHub] metron issue #905: METRON-1417: Disable pcap-service by default in Monit

2018-01-24 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/905 I one-up'd you in #903 by just removing Monit all-together. There is really no need for it any longer. It was useful before the MPack; now not so much. ---

[GitHub] metron issue #905: METRON-1417: Disable pcap-service by default in Monit

2018-01-23 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/905 Just a heads up my first manual test failed with full dev because of this guy: https://github.com/apache/metron/blob/master/metron-deployment/roles/monit-start/tasks/main.yml#L36 ``` -

[GitHub] metron issue #905: METRON-1417: Disable pcap-service by default in Monit

2018-01-23 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/905 +1, anything to save room on vagrant at the moment. ---