Merge "Fix PEP8 E265 errors"

Author: Jenkins

openstackclient/common/restapi.py

@@ -251,16 +251,6 @@ def list(self, url, data=None, response_key=None, **kwargs):
         else:
             return response.json()
 
-        ###hack this for keystone!!!
-        #data = body[response_key]
-        # NOTE(ja): keystone returns values as list as {'values': [ ... ]}
-        #           unlike other services which just return the list...
-        #if isinstance(data, dict):
-        #    try:
-        #        data = data['values']
-        #    except KeyError:
-        #        pass
-
     def set(self, url, data=None, response_key=None, **kwargs):
         """Update an object via a PUT request
 

openstackclient/compute/v2/server.py

@@ -693,7 +693,7 @@ def take_action(self, parsed_args):
             if utils.wait_for_status(
                 compute_client.servers.get,
                 server.id,
-                #callback=_show_progress,
+                callback=_show_progress,
             ):
                 sys.stdout.write('Complete\n')
             else:

openstackclient/tests/common/test_restapi.py

@@ -137,10 +137,6 @@ def test_request_get_auth(self, session_mock):
             user_agent=fake_user_agent,
         )
         gopher = api.request('GET', fake_url)
-        #session_mock.return_value.headers.setdefault.assert_called_with(
-        #    'X-Auth-Token',
-        #    fake_auth,
-        #)
         session_mock.return_value.request.assert_called_with(
             'GET',
             fake_url,

openstackclient/tests/identity/v3/fakes.py

@@ -114,7 +114,8 @@
     'description': idp_description
 }
 
-#Assignments
+# Assignments
+
 ASSIGNMENT_WITH_PROJECT_ID_AND_USER_ID = {
     'scope': {'project': {'id': project_id}},
     'user': {'id': user_id},

openstackclient/tests/volume/v1/test_volume.py

@@ -220,7 +220,6 @@ def test_volume_create_user_project_id(self):
             None,
             None,
             volume_fakes.volume_name,
-            #volume_fakes.volume_description,
             None,
             None,
             identity_fakes.user_id,
@@ -296,7 +295,6 @@ def test_volume_create_user_project_name(self):
             None,
             None,
             volume_fakes.volume_name,
-            #volume_fakes.volume_description,
             None,
             None,
             identity_fakes.user_id,

tox.ini

@@ -28,6 +28,6 @@ commands=
     python setup.py build_sphinx
 
 [flake8]
-ignore = E126,E202,W602,H302,H402,E265,H305,H307,H904
+ignore = E126,E202,W602,H302,H402,H305,H307,H904
 show-source = True
 exclude =  .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools