juju-deployer/diff-fixes-for-services-without-units-and-single-valued-cfg-tags

juju-deployer-diff-fixes-for-services-without-units-and-single-valued-cfg-tags.diff

=== modified file 'deployer/action/diff.py'
--- deployer/action/diff.py	2013-11-29 21:55:56 +0000
+++ deployer/action/diff.py	2014-10-22 16:51:05 +0000
@@ -30,7 +30,7 @@
                 'constraints'] = self.env.get_constraints(svc_name)
             self.env_state['services'][svc_name][
                 'unit_count'] = len(self.env_status[
-                    'services'][svc_name]['units'])
+                    'services'][svc_name].get('units',{}))
             rels.update(self._load_rels(svc_name))
         self.env_state['relations'] = sorted(rels)
 
@@ -123,6 +123,10 @@
     def _diff_service(self, e_s, d_s, charm):
         mod = {}
         d_sc = parse_constraints(d_s.get('constraints',''))
+        # 'tags' is a special case, as it can be multi-valued: convert to list
+        # if cfg one is a string
+        if type(d_sc.get('tags')) == str:
+            d_sc['tags'] = [d_sc['tags']]
         if d_sc != e_s['constraints']:
             mod['env-constraints'] = e_s['constraints']
             mod['cfg-constraints'] = d_sc

Until this is merged in trunk from https://code.launchpad.net/~jjo/juju-deployer/diff-fixes-for-services-without-units-and-single-valued-cfg-tags, you can apply live with:

sudo patch -d /usr/lib/python2.7/dist-packages -p0 < <(curl https://gist.githubusercontent.com/jjo/18b48de406d298ed5b60/raw/fc355f2beb8a87772fd777306d7ac861a1720c68/juju-deployer-diff-fixes-for-services-without-units-and-single-valued-cfg-tags.diff )