[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/88 --- 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-113: Project Reorganization

2016-04-25 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214487973 +1, this looks good to 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

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread dlyle65535
Github user dlyle65535 commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214457863 +1 ran on both single node Vagrant and EC2. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214416378 +1 Successfully tested PR with single node vagrant and Amazon ec2 deployments. Bravo to everyone who made this happen. It was quite an effort. --- If your

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214346871 Not sure what exactly the problem is yet. ``` TASK [metron_streaming : Copy extractor.json to ec2-52-39-96-184.us-west-2.compute.amazonaws.com] *

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread merrimanr
Github user merrimanr commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214331574 All comments so far have been addressed with the latest commit. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214301466 Docs in the various Readme.md's still refer to old naming, can we adjust those to prevent confusion? --- If your project is set up for it, you can reply to t

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214301254 It appears Metron-Topologies still exists. Is that intended? If so, can we rename to match the naming style of the other projects, please. --- If your proje

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-25 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/88#issuecomment-214295874 `deployment/vagrant/multinode-vagrant/...` [still exists in the branch](https://github.com/merrimanr/incubator-metron/tree/METRON-113/deployment). Did you i

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/88#discussion_r60856827 --- Diff: metron-deployment/roles/metron_streaming/defaults/main.yml --- @@ -0,0 +1,81 @@ +# +# Licensed to the Apache Software Foundation

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/88#discussion_r60856182 --- Diff: metron-deployment/roles/metron_streaming/defaults/main.yml --- @@ -0,0 +1,81 @@ +# +# Licensed to the Apache Software Foundation

[GitHub] incubator-metron pull request: METRON-113: Project Reorganization

2016-04-24 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/incubator-metron/pull/88 METRON-113: Project Reorganization Includes changes discussed in the project reorg dev mailing list thread. Has passed the following tests: - unit tests - integration tests -