> On Oct. 23, 2014, 11:08 p.m., Alejandro Fernandez wrote: > > dev-support/docker/docker/bin/ambaribuild.py, line 26 > > <https://reviews.apache.org/r/27103/diff/1/?file=730223#file730223line26> > > > > We should follow PEP8 style guide. functions_like_this > > variables_also_like_this > > > > And 2-space indents (even though PEP8 suggests 4)
Alejandro, thank you for your review. will fix them and re-submit. - jun ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27103/#review58157 ----------------------------------------------------------- On Oct. 23, 2014, 6:40 p.m., jun aoki wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27103/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2014, 6:40 p.m.) > > > Review request for Ambari. > > > Bugs: AMBARI-7875 > https://issues.apache.org/jira/browse/AMBARI-7875 > > > Repository: ambari > > > Description > ------- > > Run Ambari build and test on Docker container. > This way, we have more flexibility for third party libraries and build > environment. > Currently JDK 1.6 and Python 2.6 should be supported but > https://builds.apache.org/view/A-D/view/Ambari/job/Ambari-trunk-Commit/ > runs on JDK1.7, which is a discrepancy. > > > Diffs > ----- > > dev-support/docker/README.md PRE-CREATION > dev-support/docker/docker/Dockerfile PRE-CREATION > dev-support/docker/docker/bin/ambaribuild.py PRE-CREATION > dev-support/docker/docker/blueprints/single-node-HDP-2.1-blueprint1.json > PRE-CREATION > dev-support/docker/docker/blueprints/single-node-hostmapping1.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/27103/diff/ > > > Testing > ------- > > https://builds.apache.org/view/A-D/view/Ambari/job/Ambari-trunk-Commit-docker > > > Thanks, > > jun aoki > >
