Merge lp:~freyes/juju-deployer/bug-1383336 into lp:juju-deployer

Proposed by Felipe Reyes
Status: Merged
Merged at revision: 128
Proposed branch: lp:~freyes/juju-deployer/bug-1383336
Merge into: lp:juju-deployer
Diff against target: 14 lines (+2/-2)
1 file modified
deployer/service.py (+2/-2)
To merge this branch: bzr merge lp:~freyes/juju-deployer/bug-1383336
Reviewer Review Type Date Requested Status
juju-deployers Pending
Review via email: mp+238941@code.launchpad.net

Description of the change

This patch fixes a formatting problem that produces a TypeError exception.

See https://bugs.launchpad.net/juju-deployer/+bug/1383336 for more details.

To post a comment you must log in.
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

looks good, thanks

Revision history for this message
Felipe Reyes (freyes) wrote :

Hi, anything I could do to get this merged?.

Thanks,

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'deployer/service.py'
--- deployer/service.py 2014-08-15 21:04:49 +0000
+++ deployer/service.py 2014-10-20 18:58:46 +0000
@@ -92,8 +92,8 @@
92 continue92 continue
93 elif p.isdigit():93 elif p.isdigit():
94 feedback.error(94 feedback.error(
95 "Service placement to machine not supported %s to %s",95 "Service placement to machine not supported %s to %s" % (
96 self.service.name, unit_placement[idx])96 self.service.name, unit_placement[idx]))
97 elif p in services:97 elif p in services:
98 if services[p].unit_placement:98 if services[p].unit_placement:
99 feedback.error(99 feedback.error(

Subscribers

People subscribed via source and target branches