[GitHub] bigtop issue #148: BIGTOP-2548: Refresh charms for Juju 2.0 and Xenial

2016-10-12 Thread petevg
Github user petevg commented on the issue: https://github.com/apache/bigtop/pull/148 I get an error when deploying the bundle, but it may just be me. Here's what I did: 1) Edited bundle-local.yaml to fix the paths, as juju 2.0 doesn't like local:blah entries. 2) Ran the f

[GitHub] bigtop issue #148: BIGTOP-2548: Refresh charms for Juju 2.0 and Xenial

2016-10-12 Thread johnsca
Github user johnsca commented on the issue: https://github.com/apache/bigtop/pull/148 The charms must all be built and deployed as Xenial, including openjdk. However, we're also removing openjdk from the bundle, as it is now optional (for choosing a JDK other than openjdk, as openjdk

[GitHub] bigtop pull request #148: BIGTOP-2548: Refresh charms for Juju 2.0 and Xenia...

2016-10-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/bigtop/pull/148 --- 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 enab

[GitHub] bigtop issue #148: BIGTOP-2548: Refresh charms for Juju 2.0 and Xenial

2016-10-12 Thread kwmonroe
Github user kwmonroe commented on the issue: https://github.com/apache/bigtop/pull/148 Thanks for the review guys. @johnsca you're right in that this was larger than expected, mostly due to the same changes (README/metadata/layer/smoke-test) going into each of the 4 hadoop charms (NN

[GitHub] bigtop pull request #149: Update juju hadoop bundle for Juju 2.0 and Xenial

2016-10-12 Thread kwmonroe
GitHub user kwmonroe opened a pull request: https://github.com/apache/bigtop/pull/149 Update juju hadoop bundle for Juju 2.0 and Xenial - Update README with xenial and juju2 instructions. Callout juju1 equivalent commands where applicable. - Colocate apps that can live together

[GitHub] bigtop issue #149: BIGTOP-2550: Update juju hadoop bundle for Juju 2.0 and X...

2016-10-12 Thread kwmonroe
Github user kwmonroe commented on the issue: https://github.com/apache/bigtop/pull/149 ASF gitbot didn't link this PR to the jira. I think that's because I didn't title it correctly at the onset. Closing out to make a new one. --- If your project is set up for it, you can reply to

[GitHub] bigtop pull request #149: BIGTOP-2550: Update juju hadoop bundle for Juju 2....

2016-10-12 Thread kwmonroe
Github user kwmonroe closed the pull request at: https://github.com/apache/bigtop/pull/149 --- 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 en

[GitHub] bigtop pull request #150: BIGTOP-2550: Update juju hadoop bundle for Juju 2....

2016-10-12 Thread kwmonroe
GitHub user kwmonroe opened a pull request: https://github.com/apache/bigtop/pull/150 BIGTOP-2550: Update juju hadoop bundle for Juju 2.0 and Xenial - Update README with xenial and juju2 instructions. Callout juju1 equivalent commands where applicable. - Colocate apps that can

[GitHub] bigtop issue #149: BIGTOP-2550: Update juju hadoop bundle for Juju 2.0 and X...

2016-10-12 Thread kwmonroe
Github user kwmonroe commented on the issue: https://github.com/apache/bigtop/pull/149 Duped PR. See #150. --- 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,

[GitHub] bigtop issue #150: BIGTOP-2550: Update juju hadoop bundle for Juju 2.0 and X...

2016-10-12 Thread johnsca
Github user johnsca commented on the issue: https://github.com/apache/bigtop/pull/150 +1 LGTM, deployed and tested successfully on AWS. --- 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 fe