)]}'
{"cinder/tests/unit/api/contrib/test_volume_manage.py":[{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"65f96136a2c205f0142338d33aff5d7482bc5203","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        return res"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    @ddt.data({\u0027host\u0027: \u0027host_ok\u0027},"},{"line_number":205,"context_line":"              {\u0027host\u0027: \u0027user@host#backend:/vol_path+workload\u0027})"},{"line_number":206,"context_line":"    @ddt.unpack"},{"line_number":207,"context_line":"    @mock.patch(\u0027cinder.volume.api.API.manage_existing\u0027, wraps\u003dapi_manage)"},{"line_number":208,"context_line":"    def test_manage_volume_ok(self, mock_api_manage, host):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_c31bc770","line":205,"range":{"start_line":205,"start_character":51,"end_line":205,"end_character":61},"updated":"2018-05-22 15:34:56.000000000","message":"perhaps create another entry for the VMAX type scenario.\n@ddt.data({\u0027host\u0027: \u0027host_ok\u0027},\n          {\u0027host\u0027: \u0027user@host#backend:/vol_path\u0027},\n          {\u0027host\u0027: \u0027host@backend#parts+of+pool\u0027})","commit_id":"20884f12dea075899c42ab5dee0be4cd0368ec22"},{"author":{"_account_id":23601,"name":"michael","display_name":"michael-mcaleer","email":"Michael.Mcaleer@dell.com","username":"michael-mcaleer"},"change_message_id":"8e175a6568fb9b5fa0ca4c825dd9c31ce2984e5f","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        return res"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    @ddt.data({\u0027host\u0027: \u0027host_ok\u0027},"},{"line_number":205,"context_line":"              {\u0027host\u0027: \u0027user@host#backend:/vol_path+workload\u0027})"},{"line_number":206,"context_line":"    @ddt.unpack"},{"line_number":207,"context_line":"    @mock.patch(\u0027cinder.volume.api.API.manage_existing\u0027, wraps\u003dapi_manage)"},{"line_number":208,"context_line":"    def test_manage_volume_ok(self, mock_api_manage, host):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_e315a3e6","line":205,"range":{"start_line":205,"start_character":51,"end_line":205,"end_character":61},"in_reply_to":"5f7c97a3_23c07b7c","updated":"2018-05-22 15:50:22.000000000","message":"Sure no problem thanks for the feedback, didn\u0027t even occur to me at the time, change will be reflected in the next patchset","commit_id":"20884f12dea075899c42ab5dee0be4cd0368ec22"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"d9fa1779c6a3f3ddee9947d57a14a607f6ab2d12","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        return res"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    @ddt.data({\u0027host\u0027: \u0027host_ok\u0027},"},{"line_number":205,"context_line":"              {\u0027host\u0027: \u0027user@host#backend:/vol_path+workload\u0027})"},{"line_number":206,"context_line":"    @ddt.unpack"},{"line_number":207,"context_line":"    @mock.patch(\u0027cinder.volume.api.API.manage_existing\u0027, wraps\u003dapi_manage)"},{"line_number":208,"context_line":"    def test_manage_volume_ok(self, mock_api_manage, host):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_23c07b7c","line":205,"range":{"start_line":205,"start_character":51,"end_line":205,"end_character":61},"in_reply_to":"5f7c97a3_c31bc770","updated":"2018-05-22 15:42:09.000000000","message":"Good idea, this would make it more clear what values are actually used.","commit_id":"20884f12dea075899c42ab5dee0be4cd0368ec22"}]}