]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
Remove unused statements/local assignments
authorAngus Salkeld <asalkeld@redhat.com>
Mon, 19 Nov 2012 04:08:53 +0000 (15:08 +1100)
committerAngus Salkeld <asalkeld@redhat.com>
Mon, 19 Nov 2012 22:27:10 +0000 (09:27 +1100)
Change-Id: Iabc1932ef4f851567c30719de127e1165f23dd90

heat/api/cfn/v1/stacks.py
heat/api/cloudwatch/watch.py
heat/common/wsgi.py
heat/engine/resources/instance.py

index 552907b6d2813e1e3758d71ff22d034888f791a3..630dab2628e335ed9872715d5725b6100a7e6fb9 100644 (file)
@@ -118,8 +118,6 @@ class StackController(object):
             return self._stackid_format(result)
 
         con = req.context
-        parms = dict(req.params)
-
         try:
             stack_list = self.engine_rpcapi.list_stacks(con)
         except rpc_common.RemoteError as ex:
@@ -198,8 +196,6 @@ class StackController(object):
             return self._stackid_format(result)
 
         con = req.context
-        parms = dict(req.params)
-
         # If no StackName parameter is passed, we pass None into the engine
         # this returns results for all stacks (visible to this user), which
         # is the behavior described in the AWS DescribeStacks API docs
@@ -350,9 +346,6 @@ class StackController(object):
         """
 
         con = req.context
-        parms = dict(req.params)
-
-        logger.info('get_template')
         try:
             identity = self._get_identity(con, req.params['StackName'])
             templ = self.engine_rpcapi.get_template(con, identity)
@@ -381,8 +374,6 @@ class StackController(object):
         """
 
         con = req.context
-        parms = dict(req.params)
-
         try:
             templ = self._get_template(req)
         except socket.gaierror:
@@ -411,8 +402,6 @@ class StackController(object):
         Deletes the specified stack
         """
         con = req.context
-        parms = dict(req.params)
-
         try:
             identity = self._get_identity(con, req.params['StackName'])
             res = self.engine_rpcapi.delete_stack(con, identity, cast=False)
@@ -454,8 +443,6 @@ class StackController(object):
             return self._stackid_format(result)
 
         con = req.context
-        parms = dict(req.params)
-
         stack_name = req.params.get('StackName', None)
         try:
             identity = stack_name and self._get_identity(con, stack_name)
index ac6f3504f34299f03d62fb1be4c28aeef5c08211..d008f449d3cf3614add2490a44b951e5a3644e52 100644 (file)
@@ -270,7 +270,7 @@ class WatchController(object):
                     'Dimensions': dimensions}}
 
         try:
-            res = self.engine_rpcapi.create_watch_data(con, watch_name, data)
+            self.engine_rpcapi.create_watch_data(con, watch_name, data)
         except rpc_common.RemoteError as ex:
             return exception.map_remote_error(ex)
 
@@ -310,8 +310,8 @@ class WatchController(object):
 
         logger.debug("setting %s to %s" % (name, state_map[state]))
         try:
-            ret = self.engine_rpcapi.set_watch_state(con, watch_name=name,
-                                                       state=state_map[state])
+            self.engine_rpcapi.set_watch_state(con, watch_name=name,
+                                               state=state_map[state])
         except rpc_common.RemoteError as ex:
             return exception.map_remote_error(ex)
 
index dbc6a73af95bcee98a4b4bd1efcf2903811c3424..8f89628732335d1d390f19768a392aba2e8d6610 100644 (file)
@@ -493,7 +493,6 @@ class XMLResponseSerializer(object):
         # Assumption : root node is dict with single key
         root = data.keys()[0]
         eltree = etree.Element(root)
-        doc = etree.ElementTree(eltree)
         self.object_to_element(data.get(root), eltree)
         response = etree.tostring(eltree)
         logging.debug("XML response : %s" % response)
index 8364250345c4930fcd84a6e815ceaebc25dc4d5c..0d2f15f70499e4246c06161a71eeb67519e27d7c 100644 (file)
@@ -293,7 +293,6 @@ class Instance(resource.Resource):
             return
         else:
             keypairs = self.nova().keypairs.list()
-            valid_key = False
             for k in keypairs:
                 if k.name == key_name:
                     return