You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by jjmeyer0 <gi...@git.apache.org> on 2016/08/05 15:41:35 UTC
[GitHub] incubator-metron pull request #206: [Metron-352] Updated Deployment to use H...
GitHub user jjmeyer0 opened a pull request:
https://github.com/apache/incubator-metron/pull/206
[Metron-352] Updated Deployment to use HDP 2.4
Change the deployment to use the Ambari 2.2.2.0 and HDP 2.4.2.0 repositories. This included consolidating some Ansible variables into group_vars and fixing typos.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jjmeyer0/incubator-metron METRON-352
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-metron/pull/206.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #206
----
commit 3292c4fa737faae8fd506bc6b0c7428238e87270
Author: JJ <jj...@gmail.com>
Date: 2016-08-03T15:22:32Z
METRON-352 Updated deployment to use HDP 2.4. Moved some configuration to group_vars to remove duplication.
commit 00af3e47752ee86613b0bde4d8d43d7ac7e65f2c
Author: JJ <jj...@gmail.com>
Date: 2016-08-05T02:05:52Z
Fixed java home.
----
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: METRON-352 Updated Deployment to use HDP 2.4
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/206
+1 Very nice. A lot of good things cleaned-up here.
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request #206: METRON-352 Updated Deployment to use HDP...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/206
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: [Metron-352] Updated Deployment to use HDP 2.4
Posted by jjmeyer0 <gi...@git.apache.org>.
Github user jjmeyer0 commented on the issue:
https://github.com/apache/incubator-metron/pull/206
Hello, has anyone had issues with org.apache.metron.enrichment.integration.EnrichmentIntegrationTest hanging? It seems to be trhowing the same exception over and over again which cause Travis CI to kill the build. I've looked at builds that were successful. The ones I've checked also have this error, just not this many times. Also, I'm having trouble reproducing it locally. Has anyone seen this before?
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: METRON-352 Updated Deployment to use HDP 2.4
Posted by dlyle65535 <gi...@git.apache.org>.
Github user dlyle65535 commented on the issue:
https://github.com/apache/incubator-metron/pull/206
Great, I'm +1. Thanks for this.
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: METRON-352 Updated Deployment to use HDP 2.4
Posted by dlyle65535 <gi...@git.apache.org>.
Github user dlyle65535 commented on the issue:
https://github.com/apache/incubator-metron/pull/206
Outstanding. Great work! How did you test this, full_dev on Vagrant?
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: METRON-352 Updated Deployment to use HDP 2.4
Posted by jjmeyer0 <gi...@git.apache.org>.
Github user jjmeyer0 commented on the issue:
https://github.com/apache/incubator-metron/pull/206
Hello David,
Thanks for the feedback. I moved the group_vars to defaults. Also, I squashed my commits to make it easier to read. I also deleted the passwd_less_ssh role as well.
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: METRON-352 Updated Deployment to use HDP 2.4
Posted by dlyle65535 <gi...@git.apache.org>.
Github user dlyle65535 commented on the issue:
https://github.com/apache/incubator-metron/pull/206
Hi JJ,
Thanks for taking this on. Looks great! One small thing, could you look at moving the repeated group vars (e.g. hdp_stack, hdp_repo_def and the ambari variables) to defaults in the appropriate roles?
Also, since you are in the neighborhood: METRON-43 could be satisfied by removing the passwd_less_ssh role. Since it's no longer needed anyway, rather than update it to reflect these changes, how would you feel about a twofer?
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #206: METRON-352 Updated Deployment to use HDP 2.4
Posted by jjmeyer0 <gi...@git.apache.org>.
Github user jjmeyer0 commented on the issue:
https://github.com/apache/incubator-metron/pull/206
Thanks, I appreciate it. Yes, I tested this with full_dev on Vagrant. I followed this [documentation](https://cwiki.apache.org/confluence/display/METRON/Verifying+Builds).
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---