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
1=== modified file 'deployer/service.py'
2--- deployer/service.py 2014-08-15 21:04:49 +0000
3+++ deployer/service.py 2014-10-20 18:58:46 +0000
4@@ -92,8 +92,8 @@
5 continue
6 elif p.isdigit():
7 feedback.error(
8- "Service placement to machine not supported %s to %s",
9- self.service.name, unit_placement[idx])
10+ "Service placement to machine not supported %s to %s" % (
11+ self.service.name, unit_placement[idx]))
12 elif p in services:
13 if services[p].unit_placement:
14 feedback.error(

Subscribers

People subscribed via source and target branches