)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"235d057490ce4ddcb27c89a126176e29ee8651ae","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove un-used import at test_volume_transfer.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is an un-used import \u0027cinder.volume\u0027 at"},{"line_number":10,"context_line":"ciner/tests/unit/api/contrib/test_volume_transfer.py."},{"line_number":11,"context_line":"Remove it, meanwhile there is using \u0027cinder.transfer.API\u0027"},{"line_number":12,"context_line":"at line #548 and line #581, so make the import consistent"},{"line_number":13,"context_line":"by changed \u0027from cinder import transfer\u0027 to \u0027import cinder.transfer\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5afe65bd_6be1fecf","line":10,"updated":"2015-06-04 01:39:27.000000000","message":"/ciner/cinder/","commit_id":"288fd45a1eae41e3cf85e5b6974b11c4c338c7cb"}],"cinder/tests/unit/api/contrib/test_volume_transfer.py":[{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"ca372e321115b6175cea2b94e0755535155acda3","unresolved":false,"context_lines":[{"line_number":545,"context_line":"                                                            quota\u003d\u00273\u0027)"},{"line_number":546,"context_line":""},{"line_number":547,"context_line":"        self.stubs.Set("},{"line_number":548,"context_line":"            transfer.API,"},{"line_number":549,"context_line":"            \u0027accept\u0027,"},{"line_number":550,"context_line":"            fake_transfer_api_accept_throwing_VolumeSizeExceedsAvailableQuota)"},{"line_number":551,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_2f21e24d","line":548,"updated":"2015-05-28 06:37:49.000000000","message":"Can not use transfer.API because transfer is defined as a local variable below at line #553. Instead you can import cinder.transfer which will not require any changes here.","commit_id":"688fc3262960015c9f2f524df8a0cd8a8f168da4"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"11686c5010983fdadd47f782864fbe78e4a301b2","unresolved":false,"context_lines":[{"line_number":545,"context_line":"                                                            quota\u003d\u00273\u0027)"},{"line_number":546,"context_line":""},{"line_number":547,"context_line":"        self.stubs.Set("},{"line_number":548,"context_line":"            transfer.API,"},{"line_number":549,"context_line":"            \u0027accept\u0027,"},{"line_number":550,"context_line":"            fake_transfer_api_accept_throwing_VolumeSizeExceedsAvailableQuota)"},{"line_number":551,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_8ff1f66c","line":548,"in_reply_to":"7a016987_2f21e24d","updated":"2015-05-28 06:48:30.000000000","message":"Sure","commit_id":"688fc3262960015c9f2f524df8a0cd8a8f168da4"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"ca372e321115b6175cea2b94e0755535155acda3","unresolved":false,"context_lines":[{"line_number":578,"context_line":"                                                                  volume_id):"},{"line_number":579,"context_line":"            raise exception.VolumeLimitExceeded(allowed\u003d1)"},{"line_number":580,"context_line":""},{"line_number":581,"context_line":"        self.stubs.Set(transfer.API, \u0027accept\u0027,"},{"line_number":582,"context_line":"                       fake_transfer_api_accept_throwing_VolumeLimitExceeded)"},{"line_number":583,"context_line":""},{"line_number":584,"context_line":"        volume_id \u003d self._create_volume()"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_cf15aee5","line":581,"updated":"2015-05-28 06:37:49.000000000","message":"ditto","commit_id":"688fc3262960015c9f2f524df8a0cd8a8f168da4"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"72de3396cd7ca403cbc0f031517ad3c46f689329","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from cinder import exception"},{"line_number":30,"context_line":"from cinder import test"},{"line_number":31,"context_line":"from cinder.tests.unit.api import fakes"},{"line_number":32,"context_line":"import cinder.transfer"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_bedd0216","line":32,"updated":"2015-05-28 09:20:09.000000000","message":"Why are you changing\n\n    from cinder import transfer\n\nto\n\n    import cinder.transfer\n\n?","commit_id":"a1e74c90d7bc138be8b2f10f8b340ac0923df3f4"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"1f400a628d95859a52f676dec15245b9e4222698","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from cinder import exception"},{"line_number":30,"context_line":"from cinder import test"},{"line_number":31,"context_line":"from cinder.tests.unit.api import fakes"},{"line_number":32,"context_line":"import cinder.transfer"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_deebf7da","line":32,"in_reply_to":"7a016987_14cf5280","updated":"2015-05-29 03:39:33.000000000","message":"Because there is using cinder.transfer at Line #548 and Line #581. So I want to make the import consistent.","commit_id":"a1e74c90d7bc138be8b2f10f8b340ac0923df3f4"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"723287e8be06b98e0a1ae6d3da9523015fa5222d","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from cinder import exception"},{"line_number":30,"context_line":"from cinder import test"},{"line_number":31,"context_line":"from cinder.tests.unit.api import fakes"},{"line_number":32,"context_line":"import cinder.transfer"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_14cf5280","line":32,"in_reply_to":"7a016987_bedd0216","updated":"2015-05-28 11:37:42.000000000","message":"Good question.","commit_id":"a1e74c90d7bc138be8b2f10f8b340ac0923df3f4"}]}