[GitHub] [metron] tigerquoll commented on issue #1511: METRON-2256 - NodeJS testing module puppeteer should only be installed for testing

2019-09-16 Thread GitBox
tigerquoll commented on issue #1511: METRON-2256 - NodeJS testing module puppeteer should only be installed for testing URL: https://github.com/apache/metron/pull/1511#issuecomment-532039966 Blocked pending the resolution of METRON-2257

[GitHub] [metron] tigerquoll commented on issue #1509: METRON-2253 optimise npm builds towards compile time rather then runtime for dev builds

2019-09-16 Thread GitBox
tigerquoll commented on issue #1509: METRON-2253 optimise npm builds towards compile time rather then runtime for dev builds URL: https://github.com/apache/metron/pull/1509#issuecomment-532040077 Blocked pending the resolution of METRON-2257

[GitHub] [metron] tigerquoll edited a comment on issue #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds

2019-09-16 Thread GitBox
tigerquoll edited a comment on issue #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds URL: https://github.com/apache/metron/pull/1513#issuecomment-531999170 +1 (non binding) Tests run ok now This is an

[GitHub] [metron] mmiklavc commented on issue #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds

2019-09-16 Thread GitBox
mmiklavc commented on issue #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds URL: https://github.com/apache/metron/pull/1513#issuecomment-532013116 > +1 Tests run ok now Not a big deal bc this is still a draft PR, but contributors should mention "(non-binding)"

[GitHub] [metron] tigerquoll commented on issue #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds

2019-09-16 Thread GitBox
tigerquoll commented on issue #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds URL: https://github.com/apache/metron/pull/1513#issuecomment-531999170 +1 Tests run ok now This is an automated message from

[GitHub] [metron] nickwallen commented on issue #1512: METRON-2250 : Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-16 Thread GitBox
nickwallen commented on issue #1512: METRON-2250 : Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1512#issuecomment-531878215 Thanks @MohanDV ! This is working for me. I do have one concern. Rather than relying on the

[GitHub] [metron] sardell opened a new pull request #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds

2019-09-16 Thread GitBox
sardell opened a new pull request #1513: METRON-2257: Metron-Alerts GUI testing failing on MacOS builds URL: https://github.com/apache/metron/pull/1513 ## Contributor Comments This PR resolves a regression introduced in the dependency upgrades in

[GitHub] [metron] MohanDV opened a new pull request #1512: METRON-2250 : Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-16 Thread GitBox
MohanDV opened a new pull request #1512: METRON-2250 : Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1512 ## Contributor Comments This change is added as work around for https://issues.apache.org/jira/browse/AMBARI-25375 This

[GitHub] [metron] tigerquoll opened a new pull request #1511: METRON-2256 - NodeJS testing module puppeteer should only be installed for testing

2019-09-16 Thread GitBox
tigerquoll opened a new pull request #1511: METRON-2256 - NodeJS testing module puppeteer should only be installed for testing URL: https://github.com/apache/metron/pull/1511 ## Contributor Comments NodeJS Module puppeteer moved to be a dev dependency. The puppeteer module is used

[GitHub] [metron] justinleet commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-14 Thread GitBox
justinleet commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-531488660 +1, this is a great quality of life feature

[GitHub] [metron] mmiklavc commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-13 Thread GitBox
mmiklavc commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-531365909 Nice @tigerquoll! +1 from me. This is an

[GitHub] [metron] asfgit closed pull request #1510: METRON-2254 Intermittent Test Failure in RestFunctionsIntegrationTest

2019-09-13 Thread GitBox
asfgit closed pull request #1510: METRON-2254 Intermittent Test Failure in RestFunctionsIntegrationTest URL: https://github.com/apache/metron/pull/1510 This is an automated message from the Apache Git Service. To respond to

[GitHub] [metron] nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-13 Thread GitBox
nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-531229896 Thanks for the work @tigerquoll . I think this is a useful addition. +1 Let's try to get +0 or better

[GitHub] [metron] nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-13 Thread GitBox
nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#discussion_r324180085 ## File path: metron-deployment/development/centos7/ansible.cfg ## @@ -21,6

[GitHub] [metron] ottobackwards edited a comment on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-13 Thread GitBox
ottobackwards edited a comment on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501#issuecomment-531195548 https://github.com/apache/metron/pull/1261#issuecomment-486301641 It is currently conflicted,

[GitHub] [metron] ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-13 Thread GitBox
ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501#issuecomment-531195548 https://github.com/apache/metron/pull/1261#issuecomment-486301641

[GitHub] [metron] ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-13 Thread GitBox
ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501#issuecomment-531194869 I have a PR up already. The idea is to have the virtual box created and up, and have the docker container

[GitHub] [metron] ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-13 Thread GitBox
ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501#issuecomment-531194664 hahah This is an automated message

[GitHub] [metron] tigerquoll commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
tigerquoll commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-531083110 @nickwallen Great idea about verbose logging. I added a debugging section to the readme to record it.

[GitHub] [metron] tigerquoll commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
tigerquoll commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#discussion_r324021975 ## File path: metron-deployment/development/centos7/ansible.cfg ## @@ -21,6

[GitHub] [metron] tigerquoll commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-12 Thread GitBox
tigerquoll commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501#issuecomment-531080780 Heh @ottobackwards, I don't think that you can get virtual box running inside a docker container, so you

[GitHub] [metron] merrimanr commented on issue #1510: METRON-2254 Intermittent Test Failure in RestFunctionsIntegrationTest

2019-09-12 Thread GitBox
merrimanr commented on issue #1510: METRON-2254 Intermittent Test Failure in RestFunctionsIntegrationTest URL: https://github.com/apache/metron/pull/1510#issuecomment-531009815 I think this is a good solution. Thanks for finding it. +1

[GitHub] [metron] nickwallen opened a new pull request #1510: METRON-2254 Intermittent Test Failure in RestFunctionsIntegrationTest

2019-09-12 Thread GitBox
nickwallen opened a new pull request #1510: METRON-2254 Intermittent Test Failure in RestFunctionsIntegrationTest URL: https://github.com/apache/metron/pull/1510 The `RestFunctionsIntegrationTest.restGetShouldTimeoutWithSuppliedTimeout` test will intermittently fail. We use an

[GitHub] [metron] nickwallen edited a comment on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen edited a comment on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-530921717 What is also useful about this change is that it provides a means to increase the verbose-ness of

[GitHub] [metron] nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#discussion_r323859125 ## File path: metron-deployment/development/centos7/ansible.cfg ## @@ -21,6

[GitHub] [metron] nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#discussion_r323858641 ## File path: metron-deployment/development/centos6/README.md ## @@ -78,6

[GitHub] [metron] nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#discussion_r323857711 ## File path: metron-deployment/development/ubuntu14/Vagrantfile ## @@ -97,5

[GitHub] [metron] nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-530921717 What is also useful about this change is that it provides a means to increase the verbose-ness of Ansible

[GitHub] [metron] nickwallen commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-12 Thread GitBox
nickwallen commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#issuecomment-530917997 This has been merged into the feature branch: 802cbdd71a972f281a445e8025040f863987964f

[GitHub] [metron] nickwallen closed pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-12 Thread GitBox
nickwallen closed pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [metron] nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-530894011 Oh, I see in the JIRA comments that you're seeing roughly 3.5 minutes. Not monumental, but anything helps.

[GitHub] [metron] nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-12 Thread GitBox
nickwallen commented on issue #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#issuecomment-530892118 Nice idea @tigerquoll. I am for anything that improves our cycle time. How much time is this option saving in

[GitHub] [metron] mmiklavc commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-12 Thread GitBox
mmiklavc commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#issuecomment-530867322 +1 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-12 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323773999 ## File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml ## @@ -52,6 +52,10 @@

[GitHub] [metron] nickwallen closed pull request #1508: METRON-2252 PcapTopologyIntegrationTest Intermittent Failures

2019-09-12 Thread GitBox
nickwallen closed pull request #1508: METRON-2252 PcapTopologyIntegrationTest Intermittent Failures URL: https://github.com/apache/metron/pull/1508 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [metron] nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1>

2019-09-12 Thread GitBox
nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1> URL: https://github.com/apache/metron/pull/1508#issuecomment-530811457 Passed a third time... There was one failure, but it was due to us hitting the 50

[GitHub] [metron] tigerquoll commented on issue #1509: METRON-2253 optimise npm builds towards compile time rather then runtime for dev builds

2019-09-12 Thread GitBox
tigerquoll commented on issue #1509: METRON-2253 optimise npm builds towards compile time rather then runtime for dev builds URL: https://github.com/apache/metron/pull/1509#issuecomment-530796963 At the moment all un-optimised javascript files are being picked up by RAT as license

[GitHub] [metron] tigerquoll opened a new pull request #1509: METRON-2253 optimise npm builds towards compile time rather then runtime for dev builds

2019-09-12 Thread GitBox
tigerquoll opened a new pull request #1509: METRON-2253 optimise npm builds towards compile time rather then runtime for dev builds URL: https://github.com/apache/metron/pull/1509 ## Contributor Comments This PR adds a new angular build profile for npm projects called "devbuild",

[GitHub] [metron] nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1>

2019-09-11 Thread GitBox
nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1> URL: https://github.com/apache/metron/pull/1508#issuecomment-530582477 Passed a second time...

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323466506 ## File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml ## @@ -52,6 +52,10 @@

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323464592 ## File path: metron-stellar/stellar-common/pom.xml ## @@ -331,6 +331,10 @@

[GitHub] [metron] nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1>

2019-09-11 Thread GitBox
nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1> URL: https://github.com/apache/metron/pull/1508#issuecomment-530560117 First run was good...

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323451255 ## File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml ## @@ -52,6 +52,10 @@

[GitHub] [metron] nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1>

2019-09-11 Thread GitBox
nickwallen commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1> URL: https://github.com/apache/metron/pull/1508#issuecomment-530508104 @mmiklavc Sure. I've seen it pass about 3 times now as I've moved this fix across

[GitHub] [metron] mmiklavc commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1>

2019-09-11 Thread GitBox
mmiklavc commented on issue #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1> URL: https://github.com/apache/metron/pull/1508#issuecomment-530506221 Looks simple enough to me, +1. Can we kick travis a number of times on this between now and

[GitHub] [metron] nickwallen opened a new pull request #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1>

2019-09-11 Thread GitBox
nickwallen opened a new pull request #1508: Fix PcapTopologyIntegrationTest.setupAll:150->setupTopology:269 expected:<20> but was:<1> URL: https://github.com/apache/metron/pull/1508 This integration test is failing intermittently on the `feature/METRON-2088-support-hdp-3.1` feature

[GitHub] [metron] mmiklavc commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#issuecomment-530476310 Small remaining couple code comments to add, and I think this is good to go with a successful Travis build!

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323355605 ## File path: metron-platform/metron-hbase/metron-hbase-common/src/test/java/org/apache/metron/hbase/mock/MockHTable.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323353299 ## File path: metron-platform/metron-hbase/metron-hbase-common/src/test/java/org/apache/metron/hbase/mock/MockHTable.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323357378 ## File path: metron-platform/metron-hbase/metron-hbase-common/src/test/java/org/apache/metron/hbase/mock/MockHTable.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323351250 ## File path: metron-platform/metron-hbase/metron-hbase-common/src/test/java/org/apache/metron/hbase/mock/MockHTable.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323354141 ## File path: metron-platform/metron-hbase/metron-hbase-common/src/test/java/org/apache/metron/hbase/mock/MockHTable.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323345690 ## File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml ## @@ -52,6 +52,10 @@

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323344832 ## File path: metron-platform/metron-hbase-server/pom.xml ## @@ -278,6 +282,7 @@

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323309510 ## File path:

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323242629 ## File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml ## @@ -52,6 +52,10 @@

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323240960 ## File path: metron-platform/metron-hbase-server/pom.xml ## @@ -278,6 +282,7 @@

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323237011 ## File path:

[GitHub] [metron] nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r323228816 ## File path: metron-stellar/stellar-common/pom.xml ## @@ -331,6 +331,10 @@

[GitHub] [metron] anandsubbu commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
anandsubbu commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#issuecomment-530371247 pebkc... sorry about that :/. This is an automated message from the Apache Git

[GitHub] [metron] nickwallen commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
nickwallen commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#issuecomment-530367603 @anandsubbu To get the configuration back out you would need to run the following. "yaf" needs to be a string. ``` CONFIG_GET("ENRICHMENT",

[GitHub] [metron] anandsubbu commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-11 Thread GitBox
anandsubbu commented on issue #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#issuecomment-530362895 Hi @nickwallen, I ran this up on full dev. I am not able to execute the **Legacy HBase Adapter** step since there is a config issue apparently.

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on issue #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-11 Thread GitBox
JonZeolla commented on issue #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#issuecomment-530355590 +1 This is an automated message from the

[GitHub] [metron-bro-plugin-kafka] ottobackwards commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-11 Thread GitBox
ottobackwards commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r32319 ## File path: tests/Baseline/kafka.resolved-topic-override-and-config/output

[GitHub] [metron-bro-plugin-kafka] ottobackwards commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-11 Thread GitBox
ottobackwards commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323177881 ## File path: tests/Baseline/kafka.resolved-topic-override-and-config/output

[GitHub] [metron] sardell opened a new pull request #1507: METRON-2251: Update with the latest master branch

2019-09-11 Thread GitBox
sardell opened a new pull request #1507: METRON-2251: Update with the latest master branch URL: https://github.com/apache/metron/pull/1507 ## Contributor Comments This change merges the latest from master into the `feature/METRON-1856-parser-aggregation` feature branch. Because

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323012133 ## File path: tests/Baseline/kafka.resolved-topic-override-and-config/output

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323011857 ## File path: docker/run_end_to_end.sh ## @@ -196,58 +206,58 @@ rc=$?; if

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323011168 ## File path: tests/kafka/resolved-topic-override-and-config.bro ## @@

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323011602 ## File path: docker/run_end_to_end.sh ## @@ -196,58 +206,58 @@ rc=$?; if

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323011538 ## File path: tests/kafka/resolved-topic-override-only.bro ## @@ -0,0

[GitHub] [metron-bro-plugin-kafka] JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
JonZeolla commented on a change in pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36#discussion_r323009365 ## File path: docker/run_end_to_end.sh ## @@ -68,6 +69,15 @@ for i in

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-10 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r322977881 ## File path:

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-10 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r322979944 ## File path: metron-stellar/stellar-common/pom.xml ## @@ -331,6 +331,10 @@

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-10 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r322978557 ## File path:

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-10 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r322980605 ## File path: metron-platform/metron-hbase/metron-hbase-common/src/test/java/org/apache/metron/hbase/mock/MockHTable.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-10 Thread GitBox
mmiklavc commented on a change in pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506#discussion_r322981358 ## File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml ## @@ -52,6 +52,10 @@

[GitHub] [metron] nickwallen opened a new pull request #1506: METRON-2188 Upgrade to HBase 2.0.2

2019-09-10 Thread GitBox
nickwallen opened a new pull request #1506: METRON-2188 Upgrade to HBase 2.0.2 URL: https://github.com/apache/metron/pull/1506 This upgrades the feature branch `feature/METRON-2088-support-hdp-3.1` to HBase 2.0.2. The scope of the changes include: * Replace `HBaseTestingUtility`

[GitHub] [metron] ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-10 Thread GitBox
ottobackwards commented on issue #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501#issuecomment-530036173 @tigerquoll, for my issue, i was going to take it a step further and just do a ROOT level docker file, and

[GitHub] [metron-bro-plugin-kafka] ottobackwards opened a new pull request #36: METRON-2069 Add btests for bro plugin topic_name selection

2019-09-10 Thread GitBox
ottobackwards opened a new pull request #36: METRON-2069 Add btests for bro plugin topic_name selection URL: https://github.com/apache/metron-bro-plugin-kafka/pull/36 - add btests for different configuration options - add docker run_end_to_end option to surpess pcaps so you can just

[GitHub] [metron] mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-10 Thread GitBox
mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#issuecomment-529947210 Closing as it looks like this will not be sufficient to fix the issue. Fixing this issue will require an

[GitHub] [metron] mmiklavc closed pull request #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-10 Thread GitBox
mmiklavc closed pull request #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505 This is an automated message from the Apache Git Service. To

[GitHub] [metron] mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-09 Thread GitBox
mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#issuecomment-529617771 @MohanDV I'm a bit surprised a "recommendation" is being used for more than a recommendation. Sounds like it's

[GitHub] [metron] MohanDV edited a comment on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-09 Thread GitBox
MohanDV edited a comment on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#issuecomment-529450070 @mmiklavc I have tested this on 5 node cluster. This workaround will not solve the issue, this will

[GitHub] [metron] MohanDV commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-09 Thread GitBox
MohanDV commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#issuecomment-529450070 @mmiklavc I have tested this on 5 node cluster. This workaround will not solve the issue, this will just

[GitHub] [metron] justinleet commented on a change in pull request #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-08 Thread GitBox
justinleet commented on a change in pull request #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#discussion_r322015747 ## File path:

[GitHub] [metron] JonZeolla commented on issue #1282: METRON-1881: Update the bro plugin used in metron development

2019-09-07 Thread GitBox
JonZeolla commented on issue #1282: METRON-1881: Update the bro plugin used in metron development URL: https://github.com/apache/metron/pull/1282#issuecomment-529101714 In my opinion, yes. This is an automated message from

[GitHub] [metron] tigerquoll opened a new pull request #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-06 Thread GitBox
tigerquoll opened a new pull request #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501 ## Contributor Comments Docker bind shared folders have a reputation as being fairly slow and inefficient on Docker for

[GitHub] [metron] tigerquoll closed pull request #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance

2019-09-06 Thread GitBox
tigerquoll closed pull request #1501: METRON-2246 rpm-docker - minimise use of bind mounts due to performance URL: https://github.com/apache/metron/pull/1501 This is an automated message from the Apache Git Service. To

[GitHub] [metron] tigerquoll opened a new pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-06 Thread GitBox
tigerquoll opened a new pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503 ## Contributor Comments This PR allows full dev deploy users to skip rpmlint processing via setting an environment variable.

[GitHub] [metron] tigerquoll closed pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint

2019-09-06 Thread GitBox
tigerquoll closed pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503 This is an automated message from the Apache Git Service. To

[GitHub] [metron] mmiklavc commented on issue #1503: Metron-2247

2019-09-06 Thread GitBox
mmiklavc commented on issue #1503: Metron-2247 URL: https://github.com/apache/metron/pull/1503#issuecomment-529008256 @tigerquoll please update the PR title - the git elves won't match on mixed-case for the Jira logging. Needs to be "METRON-2247 rpm-docker: Provide an option to bypass

[GitHub] [metron] mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-06 Thread GitBox
mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#issuecomment-528958112 @anandsubbu @MohanDV - would be great if you could take a look at this in a multi-node cluster and confirm it

[GitHub] [metron] mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-06 Thread GitBox
mmiklavc commented on issue #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505#issuecomment-528957820 ### Test Plan In order reproduce the error being fixed in this bug, pull the latest feature branch code

[GitHub] [metron] mmiklavc opened a new pull request #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-06 Thread GitBox
mmiklavc opened a new pull request #1505: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1505 ## Contributor Comments https://issues.apache.org/jira/browse/METRON-2250 Workaround for a bug in the Ambari. See

[GitHub] [metron] mmiklavc commented on issue #1504: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-06 Thread GitBox
mmiklavc commented on issue #1504: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1504#issuecomment-528952698 Wrong branch This is an automated message

[GitHub] [metron] mmiklavc closed pull request #1504: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-06 Thread GitBox
mmiklavc closed pull request #1504: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1504 This is an automated message from the Apache Git Service. To

[GitHub] [metron] mmiklavc opened a new pull request #1504: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck

2019-09-06 Thread GitBox
mmiklavc opened a new pull request #1504: METRON-2250 Missing services in HDP 3.1 metron mpack and installer stuck URL: https://github.com/apache/metron/pull/1504 ## Contributor Comments https://issues.apache.org/jira/browse/METRON-2250 Workaround for a bug in the Ambari. See

[GitHub] [metron] nickwallen closed pull request #1502: METRON-2248 Merge Master into Feature Branch

2019-09-06 Thread GitBox
nickwallen closed pull request #1502: METRON-2248 Merge Master into Feature Branch URL: https://github.com/apache/metron/pull/1502 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [metron] nickwallen edited a comment on issue #1502: METRON-2248 Merge Master into Feature Branch

2019-09-06 Thread GitBox
nickwallen edited a comment on issue #1502: METRON-2248 Merge Master into Feature Branch URL: https://github.com/apache/metron/pull/1502#issuecomment-528885930 This has been merged into ~master~ the feature branch. This is

<    4   5   6   7   8   9   10   11   12   13   >