]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
Catch errors better when creating the stack
authorAngus Salkeld <asalkeld@redhat.com>
Fri, 16 Mar 2012 01:09:42 +0000 (12:09 +1100)
committerAngus Salkeld <asalkeld@redhat.com>
Fri, 16 Mar 2012 03:14:25 +0000 (14:14 +1100)
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
heat/api/v1/stacks.py

index 3eb1a4712bdef775bd17684d83b3d305cae78f1a..483119dba074daf38a3ab561959549bcedcb5962 100644 (file)
@@ -121,9 +121,12 @@ class Json2CapeXml:
                 # if there is no config then no services.
                 pass
 
-        filename = '/var/run/%s.xml' % name
-        open(filename, 'w').write(doc.serialize(None, 1))
-        doc.freeDoc()
+        try:
+            filename = '/var/run/%s.xml' % name
+            open(filename, 'w').write(doc.serialize(None, 1))
+            doc.freeDoc()
+        except IOError as e:
+            logger.error('couldn\'t write to /var/run/ error %s' % e)
 
     def insert_package_and_services(self, r, new_script):
 
@@ -259,7 +262,11 @@ def systemctl(method, name, instance=None):
     else:
         service = '%s@%s.service' % (name, instance)
 
-    result = m(service, 'replace')
+    try:
+        result = m(service, 'replace')
+    except dbus.DBusException as e:
+        logger.error('couldn\'t %s %s error: %s' % (method, name, e))
+        return None
     return result