)]}'
{"mistral/actions/openstack/actions.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"e2a01d97c8c76240c65108eca5be6b34a680e752","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from oslo_log import log"},{"line_number":19,"context_line":"from oslo_utils import importutils"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from keystoneauth1.identity import v3"},{"line_number":22,"context_line":"from keystoneauth1 import session as ks_session"},{"line_number":23,"context_line":"from keystoneauth1.token_endpoint import Token"},{"line_number":24,"context_line":"from keystoneclient import httpclient"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_50ab2a88","line":21,"range":{"start_line":21,"start_character":35,"end_line":21,"end_character":37},"updated":"2019-09-10 04:50:10.000000000","message":"Not the best way to import modules. This name (v3) is too generic. When I first saw \"v3\" in the code I had no idea what it referred to. I\u0027d do something like \".. import v3 as ks_identity_v3\"","commit_id":"76d03550fd7e17137352c81c970c1398ca9e0f22"}]}