Automate creation of bigtop/slaves docker images.
Provide JDK8 for Bigtop
Thanks to open up the jira. Just want to let you know how I built the images now.
Currently I'm using Jenkins job to create slaves images and push them up to the docker hub manually: http://bigtop01.cloudera.org:8080/view/Docker/job/Docker-Toolchain-Trunk/
It'd be better to wrap them as Dockerfiles and puts under bigtop/docker/bigtop-slaves directory.
Yep Evans Ye thanks for pointing me to the missing piece!
Regarding the naming as of BIGTOP-1953: Yep, should be better to include "trunk" for git trunk and the branch name for branches. Will include in my WIP
Uploaded WIP patch.
You may be curious why I include puppet_modules in this patch.
I need to integrate openjdk8 in BIGTOP-1936, located in debian-backports. I am using apt::backports to add into sources
Added a last minute change to support adding PPA to ubuntu
Support for fedora-20 as well.
"official" Docker image is broken, need to enable fedora-updates-testing, since one gets conflicts for openssl and kerberos.
Evans Ye : Sorry for the constant stream of updates. This should be final.
Looks good. One small concern. Is it better to combine all 3 RUN commands into one striped command? In that way we reduce the layers of docker images.
Removing on of the docker aufs layers, two for fedora.
I have no idea how I can merge all RUN commands to one, since the target directory of the COPY has to be created first and the second RUN is dependent of the COPY.
You can do that since docker will put the directory/file in the right place anyway:
COPY bigtop_toolchain /etc/puppet/modules/bigtop_toolchain
RUN rpm -ivh http://yum.puppetlabs.com/puppetlabs-release-el-6.noarch.rpm; \
yum -y install curl sudo unzip wget puppet tar; \
puppet apply -e "include bigtop_toolchain::puppet-modules" ; \
puppet apply -e "include bigtop_toolchain::installer"
However, it's a minor change so I'll let you decide.
Here's my +1 for the last patch, its good!
Indeed, wil change the files again.
Patch edited, uploaded and tested. Ok, to commit ?
Eeek, opensuse had a cut&paste error, fixed
+1. I've committed this. Thanks for following up with my comment Olaf Flebbe.