Merge "Fix PEP H902 (Use the 'not in' operator)"

diff --cc heat/engine/parser.py
index 2d5a8c4c462833ae4306a034030b773de6840ea5,fc30a4e23823cd335cee8d6987c997a069bfa569..624d7f7835b7f7d71272cf844d6d6b3f39df2927
--- 1/heat/engine/parser.py
--- 2/heat/engine/parser.py
+++ b/heat/engine/parser.py
@@@ -349,14 -349,20 +349,14 @@@ class Stack(object)
              try:
                  # First delete any resources which are not in newstack
                  for res in reversed(self):
-                     if not res.name in newstack.keys():
+                     if res.name not in newstack.keys():
                          logger.debug("resource %s not found in updated stack"
                                       % res.name + " definition, deleting")
 -                        try:
 -                            res.destroy()
 -                        except exception.ResourceFailure as ex:
 -                            logger.error("Failed to remove %s : %s" %
 -                                         (res.name, str(ex)))
 -                            raise exception.ResourceUpdateFailed(
 -                                resource_name=res.name)
 -                        else:
 -                            del self.resources[res.name]
 -                            self.dependencies = self._get_dependencies(
 -                                self.resources.itervalues())
 +                        # res.destroy raises exception.ResourceFailure on error
 +                        res.destroy()
 +                        del self.resources[res.name]
 +                        self.dependencies = self._get_dependencies(
 +                            self.resources.itervalues())
  
                  # Then create any which are defined in newstack but not self
                  for res in newstack: