Code review comment for lp://staging/~gary/juju-gui/bug1218924

Revision history for this message
Gary Poster (gary) wrote :

*** Submitted:

Change distfile to exclude unnecessary bits

In order to address the charm deployment problems reported today, this
is a first step of two that significantly shrinks the size of our
releases. It takes our release down from about 43M to 9M; that brings
our charm from about 45M to 11M. I'll make another step (switch from gz
to xz) that can bring us down another couple of M. Hopefully that will
be sufficient for the problem.

The code is pretty easy to review, but the qa is a bit more tedious.
First, please make a distfile (BRANCH_IS_GOOD=1 make distfile) and then
follow the instructions I added to the release instructions to test the
resulting tarball. Second, add the tarball to a local copy of the charm
in the releases directory, remove the 0.10.1 tarball, juju bootstrap,
and run make deploy in the charm. Once the charm is deployed, you
should be able to see qa the site.

R=frankban
CC=
https://codereview.appspot.com/14516054

https://codereview.appspot.com/14516054/

« Back to merge proposal