)]}'
{"openstack_dashboard/api/glance.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"3891b643fc96d6dc32598d098a4de0956ba8ba35","unresolved":false,"context_lines":[{"line_number":52,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":53,"context_line":"VERSIONS \u003d base.APIVersionManager(\"image\", preferred_version\u003d2)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"try:"},{"line_number":56,"context_line":"    # pylint: disable\u003dungrouped-imports"},{"line_number":57,"context_line":"    from glanceclient.v2 import client as glance_client_v2"},{"line_number":58,"context_line":"    VERSIONS.load_supported_version(2, {\"client\": glance_client_v2,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_225e5712","line":55,"range":{"start_line":55,"start_character":0,"end_line":55,"end_character":4},"updated":"2019-10-05 17:47:27.000000000","message":"I think we can drop this try/except as we now depend on glanceclient.v2.","commit_id":"d32ff36739ecd4083dabb215012b08030957d04b"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"3891b643fc96d6dc32598d098a4de0956ba8ba35","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"try:"},{"line_number":56,"context_line":"    # pylint: disable\u003dungrouped-imports"},{"line_number":57,"context_line":"    from glanceclient.v2 import client as glance_client_v2"},{"line_number":58,"context_line":"    VERSIONS.load_supported_version(2, {\"client\": glance_client_v2,"},{"line_number":59,"context_line":"                                        \"version\": 2})"},{"line_number":60,"context_line":"except ImportError:"},{"line_number":61,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_02b8fb27","line":58,"range":{"start_line":57,"start_character":4,"end_line":58,"end_character":67},"updated":"2019-10-05 17:47:27.000000000","message":"We depend on glanceclient v2, so L.57 would be unnecessary and glance_client_v2 in L.58 can be changed to \"glance_client.v2.client\" (note that \"glance_client\" is imported at L.33.)","commit_id":"d32ff36739ecd4083dabb215012b08030957d04b"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"9f5e78fc220fa0377cb9f8ee8c16dd7ff654d046","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"try:"},{"line_number":56,"context_line":"    # pylint: disable\u003dungrouped-imports"},{"line_number":57,"context_line":"    from glanceclient.v2 import client as glance_client_v2"},{"line_number":58,"context_line":"    VERSIONS.load_supported_version(2, {\"client\": glance_client_v2,"},{"line_number":59,"context_line":"                                        \"version\": 2})"},{"line_number":60,"context_line":"except ImportError:"},{"line_number":61,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_77f87348","line":58,"range":{"start_line":57,"start_character":4,"end_line":58,"end_character":67},"in_reply_to":"3fa7e38b_02b8fb27","updated":"2019-10-08 09:36:42.000000000","message":"Sounds reasonable. I will change it in a follow up patch","commit_id":"d32ff36739ecd4083dabb215012b08030957d04b"}]}