Use TODO properly, remove Tempest output

Author: Isaac Karrer

jumpgate/compute/drivers/sl/extensions.py

@@ -9,6 +9,7 @@
         'namespace': 'http://docs.openstack.org/compute/ext/availabilityzone/'
                      'api/v1.1',
         'updated': '2012-12-21T00:00:00+00:00',
+        # TODO(imkarrer) Added availability zone, need to return real value
         'availability_zone': {'Hosts': None},
     },
 }

jumpgate/compute/drivers/sl/servers.py

@@ -213,7 +213,7 @@ def on_post(self, req, resp, tenant_id):
         resp.set_header('x-compute-request-id', 'create')
         resp.status = 202
         resp.body = {'server': {
-            # TODO(imkarrer) - Casted to string to make tempest pass, make sure correct  # noqa
+            # Casted to string to make tempest pass
             'id': str(new_instance['id']),
             'links': [{
                 'href': self.app.get_endpoint_url(
@@ -560,8 +560,8 @@ def get_server_details_dict(app, req, instance, is_list):
 
     # TODO(kmcdonald) - Don't hardcode this
     image_name = ''
-    # TODO(imkarrer) - returning None makes tempest fail, conditionally returning empty string  # noqa
-    # TODO(imkarrer) - same thing with zone
+    # returning None makes tempest fail,
+    # conditionally returning empty string for uid and zone
     uid = str(utils.lookup(instance,
                            'billingItem',
                            'orderItem',
@@ -608,7 +608,6 @@ def get_server_details_dict(app, req, instance, is_list):
         'tenant_id': tenant_id,
         # NOTE(bodenr): userRecordId accessibility determined by permissions
         # of API caller's user id and api key. Otherwise it will be None
-        # TODO(imkarrer) - returning None makes tempest fail, conditionally returning empty string  # noqa
         'user_id': uid,
         'updated': instance['modifyDate'],
         'image_name': image_name,

jumpgate/identity/drivers/sl/auth_tokens_v3.py

@@ -39,7 +39,7 @@ def parse_templates(template_lines):
 def get_access(token_id, token_details, user):
     return {
         'token': {
-            # TODO(imkarrer) - replaced isoformat() with strftime to make tempest pass  # noqa
+            # replaced isoformat() with strftime to make tempest pass
             'expires': datetime.datetime.fromtimestamp(
                 token_details['expires']).strftime('%Y-%m-%dT%H:%M:%SZ'),
             'id': token_id,

jumpgate/identity/drivers/sl/tokens.py

@@ -41,7 +41,7 @@ def get_access(token_id, token_details):
     tokens = identity.token_driver()
     return {
         'token': {
-            # TODO(imkarrer) - replaced isoformat() with strftime to make tempest pass  # noqa
+            # replaced isoformat() with strftime to make tempest pass
             'expires': datetime.datetime.fromtimestamp(
                 tokens.expires(token_details)).strftime('%Y-%m-%dT%H:%M:%SZ'),
             'id': token_id,

jumpgate/image/drivers/sl/images.py

@@ -570,7 +570,7 @@ def get_v1_image_details_dict(app, req, image, tenant_id=None):
         'progress': 100,
         'metadata': {},
         'size': int(image.get('blockDevicesDiskSpaceTotal', 0)),
-        # TODO() - changed from None to 1000 for Tempest
+        # changed from None to 1000 for Tempest
         'OS-EXT-IMG-SIZE:size': 1000,
         'container_format': 'bare',
         'disk_format': 'raw',