Merge lp://staging/~ev/ubuntu-ci-services-itself/pin-jujuclient into lp://staging/ubuntu-ci-services-itself

Proposed by Evan
Status: Merged
Approved by: Chris Johnston
Approved revision: 310
Merged at revision: 312
Proposed branch: lp://staging/~ev/ubuntu-ci-services-itself/pin-jujuclient
Merge into: lp://staging/ubuntu-ci-services-itself
Diff against target: 25 lines (+9/-0)
1 file modified
juju-deployer/deploy.py (+9/-0)
To merge this branch: bzr merge lp://staging/~ev/ubuntu-ci-services-itself/pin-jujuclient
Reviewer Review Type Date Requested Status
Chris Johnston (community) Approve
Andy Doan (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+209345@code.staging.launchpad.net

Commit message

Pin python-jujuclient to r24, working around bug 1284183 and fixing bug 1287740.

Description of the change

Pin python-jujuclient to r24, working around bug 1284183:

"for folks running into this via deployer, note that jujuclient 0.17.3 (in trusty or installable via pypi) works around the api server disconnecting all clients by auto reconnecting for watches."

This also fixes bug 1287740.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:310
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/293/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/293/rebuild

review: Approve (continuous-integration)
Revision history for this message
Andy Doan (doanac) wrote :

i love MP's that improve the quality of my life.

review: Approve
Revision history for this message
Evan (ev) wrote :

On 4 March 2014 23:12, Andy Doan <email address hidden> wrote:
> Review: Approve
>
> i love MP's that improve the quality of my life.

LOL! That just made my night.

Revision history for this message
Chris Johnston (cjohnston) wrote :

On Tue, Mar 4, 2014 at 6:21 PM, Evan Dandrea <email address hidden>wrote:

> On 4 March 2014 23:12, Andy Doan <email address hidden> wrote:
> > Review: Approve
> >
> > i love MP's that improve the quality of my life.
>
> LOL! That just made my night.
>
> Wow... must be a rockin night over there.

 merge approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: