[GitHub] incubator-metron pull request: METRON-82 Enhance Travis CI Builds

2016-03-21 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/incubator-metron/pull/56 METRON-82 Enhance Travis CI Builds - Display current CI build status in README. - Added ability to run CI against multiple JVMs. You can merge this pull request into a Git repositor

[GitHub] incubator-metron pull request: METRON-81: Metron-UI Deployment sho...

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/55 --- 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 featur

[GitHub] incubator-metron pull request: METRON-81: Metron-UI Deployment sho...

2016-03-21 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/55#issuecomment-199554897 +1. This should avoid pulling over local copies node modules, which we're seeing periodically. --- If your project is set up for it, you can reply to this em

[GitHub] incubator-metron pull request: METRON-81: Metron-UI Deployment sho...

2016-03-21 Thread dlyle65535
GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/55 METRON-81: Metron-UI Deployment should exclude node_modules Excluded node_modules from the metron-ui tar call. You can merge this pull request into a Git repository by running: $ gi

[GitHub] incubator-metron pull request: METRON-79 PcapService is hanging wh...

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/54 --- 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 featur

[GitHub] incubator-metron pull request: METRON-79 PcapService is hanging wh...

2016-03-21 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/54#issuecomment-199471482 +1 from me --- 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 f

[GitHub] incubator-metron pull request: METRON-79 PcapService is hanging wh...

2016-03-21 Thread dlyle65535
Github user dlyle65535 commented on the pull request: https://github.com/apache/incubator-metron/pull/54#issuecomment-199470957 +1. Ran this on EC2 and verified pcap was searchable from the UI. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] incubator-metron pull request: METRON-79 PcapService is hanging wh...

2016-03-21 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/incubator-metron/pull/54 METRON-79 PcapService is hanging when called from the UI This pull request includes changes to the pcapservice Ansible script so that the hbase-site.xml file is put in the correct locatio

[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/53 --- 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 featur

[GitHub] incubator-metron pull request: METRON-73 Sensor Service Wrappers a...

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/49 --- 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 featur

[GitHub] incubator-metron pull request: [METRON-19] Enable Travis CI testin...

2016-03-21 Thread ehashman
Github user ehashman commented on the pull request: https://github.com/apache/incubator-metron/pull/9#issuecomment-199368689 I don't think @reaperhulk and I have the bandwidth to get this working; please feel free to branch off our work and add some commits. --- If your project is se

[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...

2016-03-21 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/53#issuecomment-199339197 +1 from me now. I believe the travis error is due to travis not being set up yet, which will happen external to this PR. --- If your project is set up for it

[GitHub] incubator-metron pull request: [METRON-19] Enable Travis CI testin...

2016-03-21 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/9#issuecomment-199334324 Also, Travis fails the builds because the output exceeds 4MB. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...

2016-03-21 Thread dlyle65535
Github user dlyle65535 commented on the pull request: https://github.com/apache/incubator-metron/pull/53#issuecomment-199327208 No thanks @merrimanr. I had that wrong. The regexes match on the lhs and replace the rhs. It works like a champ. --- If your project is set up for it, you c

[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...

2016-03-21 Thread merrimanr
Github user merrimanr commented on the pull request: https://github.com/apache/incubator-metron/pull/53#issuecomment-199326697 Dave commented in Metron-72 that we should revert /incubator-metron/metron-streaming/Metron-Topologies/src/main/resources/Metron_Configs/etc/env/config.propert

[GitHub] incubator-metron pull request: Metron 73 - Sensor Service Wrappers...

2016-03-21 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/49#issuecomment-199325968 PR is good. Travis CI check failed because we do not have a good .travis.yml yet. --- If your project is set up for it, you can reply to this email and hav

Re: CI Builds

2016-03-21 Thread Nick Allen
Travis CI has been enabled by the Apache Infrastructure team. On Mon, Mar 21, 2016 at 9:59 AM, Nick Allen wrote: > Submitted a request to Apache to at least enable Travis CI connectivity > with Github. > > https://issues.apache.org/jira/browse/INFRA-11510 > > We can work out the details of what

[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...

2016-03-21 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/53#issuecomment-199292568 This looks good. I especially like the single-node deployment only running bro, snort, yaf and enrichment. I think we could even trim it down to just bro and

[GitHub] incubator-metron pull request: [METRON-19] Enable Travis CI testin...

2016-03-21 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/9#issuecomment-199296701 This looks really good. A few other changes needed. We will need to (1) add the apache-rat license check (2) execute the 'integration-test' target and (3) f

Re: CI Builds

2016-03-21 Thread Nick Allen
Submitted a request to Apache to at least enable Travis CI connectivity with Github. https://issues.apache.org/jira/browse/INFRA-11510 We can work out the details of what actually happens once the Apache request is complete. On Mon, Mar 21, 2016 at 9:41 AM, Nick Allen wrote: > Ah, yes. That

[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...

2016-03-21 Thread dlyle65535
GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/53 METRON-77: Refactor Deployment Scripts to Accept Changes from METRON-72 PCAP topology filename is now remote.yaml. YAF indexing can sometimes auto-code risn as a number. Metron-UI

Re: CI Builds

2016-03-21 Thread Nick Allen
Ah, yes. That work looks good, although a few other things have changed since then. We need to (1) add the apache-rat license check (2) execute the 'integration-test' target. Would you be willing to update that PR or submit a new one? If so, I will try and shepard in your changes. On Mon, Ma

Re: CI Builds

2016-03-21 Thread Elana Hashman
Paul and I worked on this two months ago but it never got merged: https://github.com/apache/incubator-metron/pull/9 I'd like to get automated CI builds working for the project. It seems that Travis CI (https://travis-ci.org) is quite popular and integrates well with Github. I would certainly b

[GitHub] incubator-metron pull request:

2016-03-21 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/commit/68aab6e9adc42b3a21701d1fca2c1a3a81e14237#commitcomment-16783360 In deployment/roles/httplib2/tasks/main.yml: In deployment/roles/httplib2/tasks/main.yml on line 18: Why have

[GitHub] incubator-metron pull request: Minor typos in Howtos

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/52 --- 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 featur

[GitHub] incubator-metron pull request: Minor typos in Howtos

2016-03-21 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/52#issuecomment-199264783 +1, this is great. Thank you very much for your contribution! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-metron pull request: Minor typos in Howtos

2016-03-21 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/52#issuecomment-199263377 +1 Thanks! --- 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