LGTM. Although it smells like version.Current was leaking in there somewhere if upgrading the host caused tests to fail.
On Thu, Apr 17, 2014 at 11:57 AM, Tim Penhey <email address hidden> wrote: > LGTM > > although we really should get better at isolating this. > > https://codereview.appspot.com/88730043/ > > -- > https://code.launchpad.net/~axwalk/juju-core/bootstrap-test-tools/+merge/216237 > You are subscribed to branch lp:juju-core.
« Back to merge proposal
LGTM. Although it smells like version.Current was leaking in there
somewhere if upgrading the host caused tests to fail.
On Thu, Apr 17, 2014 at 11:57 AM, Tim Penhey <email address hidden> wrote: /codereview. appspot. com/88730043/ /code.launchpad .net/~axwalk/ juju-core/ bootstrap- test-tools/ +merge/ 216237
> LGTM
>
> although we really should get better at isolating this.
>
> https:/
>
> --
> https:/
> You are subscribed to branch lp:juju-core.