[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 #904: METRON-1400: Elasticsearch service check fails in Ambari

2018-01-23 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/904 Travis failure appears unrelated ---

[GitHub] metron issue #904: METRON-1400: Elasticsearch service check fails in Ambari

2018-01-23 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/904 +1 by inspection, thanks! ---

[GitHub] metron pull request #905: METRON-1417: Disable pcap-service by default in Mo...

2018-01-23 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/metron/pull/905 METRON-1417: Disable pcap-service by default in Monit ## Contributor Comments https://issues.apache.org/jira/browse/METRON-1417 Free up some full dev mem. We don't run the pcap

[GitHub] metron pull request #904: METRON-1400: Elasticsearch service check fails in ...

2018-01-23 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/metron/pull/904 METRON-1400: Elasticsearch service check fails in Ambari ## Contributor Comments Addresses https://issues.apache.org/jira/browse/METRON-1400 This file hasn't been touched since at

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

[GitHub] metron pull request #903: METRON-1370 Create Full Dev Equivalent for Ubuntu

2018-01-23 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/903 METRON-1370 Create Full Dev Equivalent for Ubuntu This PR adds a single node, VM based deployment mechanism based on Ubuntu Trusty. This has feature parity with the existing CentOS

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

2018-01-23 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/853 Just ducking in here, @merrimanr is this ready for review? Specifically, the responses to @justinleet 's questions have all been factored into the current code for this PR, right? If yes,

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

2018-01-23 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/853 We're good on 1-4. For point 5. I'd still like to see a note in the upgrading, even if it gets removed when the more flexible store is added. I don't know when we'll get around to using

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

2018-01-23 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/853 I don't see anything that should be problematic in ES 5.6.2, but can you confirm @merrimanr? ---

[GitHub] metron issue #903: METRON-1370 Create Full Dev Equivalent for Ubuntu

2018-01-23 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/903 I am continuing a final round of testing to make sure these changes are solid. I wanted to get the PR open so I could start getting feedback as soon as possible. ---

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

2018-01-23 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/853 I will address 6 shortly. For 5, should we explore a more flexible store in this PR? Or at least validate that an RDBMS is the right choice? I think this is something we should tackle now as it

Re: Master is failed in Travis

2018-01-23 Thread Michael Miklavcic
Yeah, this seems to be breaking in every build at this point. I am going to look into it tomorrow. On Mon, Jan 22, 2018 at 8:29 AM, Nick Allen wrote: > I had copied the wrong text into the bug. I fixed that. > > On Mon, Jan 22, 2018 at 10:22 AM, Casey Stella