)]}'
{"nova/api/opts.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"db11fd1dbb35d0e4915883379dcb5a43d99fd25f","unresolved":false,"context_lines":[{"line_number":110,"context_line":"import nova.vnc.xvp_proxy"},{"line_number":111,"context_line":"import nova.volume"},{"line_number":112,"context_line":"import nova.volume.cinder"},{"line_number":113,"context_line":"import nova.wsgi"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"def list_opts():"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_6afd2cb0","side":"PARENT","line":113,"updated":"2016-04-21 09:30:01.000000000","message":"Why are all these being dropped here? That doesn\u0027t seem quite right.","commit_id":"1f687d078e8940e01dbaf4e3179409c0bc50a807"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6c64ba5869421ef790c95d68a5184e04c602695f","unresolved":false,"context_lines":[{"line_number":110,"context_line":"import nova.vnc.xvp_proxy"},{"line_number":111,"context_line":"import nova.volume"},{"line_number":112,"context_line":"import nova.volume.cinder"},{"line_number":113,"context_line":"import nova.wsgi"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"def list_opts():"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_be7b02f4","side":"PARENT","line":113,"in_reply_to":"1a122d0e_2d706e71","updated":"2016-04-21 10:50:01.000000000","message":"OK, it just seems odd to do that in this patch, feels like an unrelated change, I guess. Its odd pep8 didn\u0027t catch this lot before.","commit_id":"1f687d078e8940e01dbaf4e3179409c0bc50a807"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"4e5b06c8b72c71191cd540289c9032e2b7d46e8d","unresolved":false,"context_lines":[{"line_number":110,"context_line":"import nova.vnc.xvp_proxy"},{"line_number":111,"context_line":"import nova.volume"},{"line_number":112,"context_line":"import nova.volume.cinder"},{"line_number":113,"context_line":"import nova.wsgi"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"def list_opts():"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_2d706e71","side":"PARENT","line":113,"in_reply_to":"1a122d0e_6afd2cb0","updated":"2016-04-21 09:49:12.000000000","message":"I have no idea why all these are placed here, so I remove all unused import","commit_id":"1f687d078e8940e01dbaf4e3179409c0bc50a807"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"6a471d631456e693f7828a3ee3c7a07cba870c45","unresolved":false,"context_lines":[{"line_number":110,"context_line":"import nova.vnc.xvp_proxy"},{"line_number":111,"context_line":"import nova.volume"},{"line_number":112,"context_line":"import nova.volume.cinder"},{"line_number":113,"context_line":"import nova.wsgi"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"def list_opts():"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_c08afd19","side":"PARENT","line":113,"in_reply_to":"1a122d0e_be7b02f4","updated":"2016-04-23 23:16:06.000000000","message":"It looks like flake8 that we have only detects unused imports of the form\n\nfrom a import b\n\nimport a.b will pass\n\nSo it is unrelated, but fine to clean up. Ideally I\u0027d rather see something like this in a separate patch in future.","commit_id":"1f687d078e8940e01dbaf4e3179409c0bc50a807"}]}