[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 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 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-11 Thread johnsca
Github user johnsca commented on the issue: https://github.com/apache/bigtop/pull/148 This is a pretty big PR, but it looks good. I was able to build and deploy the updated charms and run the smoke-tests. The README changes look good. The commits do need to be squashed, thou