/layer-apache-bigtop-base forked from c0s/juju-apache-bigtop

Added option to have Bigtop handle java installation. #39

Closed
wants to merge 3 commits into
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@petevg @johnsca
@petevg

petevg commented Aug 15, 2016

It is turned off by default.

Question for review: is a build time config right for this? Or should we allow operators to specify in the config?

@juju-solutions/bigdata

petevg added some commits Aug 10, 2016

@petevg
Added option to have Bigtop handle java installation. 
It is turned off by default.
c61efa4
@petevg
Fixed stray reference to charms.unit. 
We still need to be using bigtop_harness.
936c716
@johnsca
Owner

johnsca commented Aug 16, 2016

Hrm. I was not envisioning an all-or-nothing approach using a build-time flag like this. Do we not think that it's possible to start off with Bigtop managing Java and overriding it after the fact if / when the Java relation is applied?
@petevg
Turned on Bigtop java by default. 
We should still use the openjdk relation if it gets setup, but we want
to have Bigtop install java by default from now on.
4e01ca4

@petevg petevg closed this Aug 25, 2016

@petevg petevg deleted the feature/auto_install_java branch Aug 25, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment