)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2861,"name":"Vincent Hou","email":"shou@us.ibm.com","username":"houshengbo"},"change_message_id":"d7ca84a6523fd1cf1b983832a223dddaa02a48ec","unresolved":false,"context_lines":[{"line_number":15,"context_line":"management of volumes that are not currently mapped to a server on the"},{"line_number":16,"context_line":"Storage Center. id-types of source-id (DeviceId on the Storage Center)"},{"line_number":17,"context_line":"and source-name (Volume Name on the Storage Center) are supported."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Iecc93ce1654111e1292cfc1931d1eda4abd08595"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba3cc151_88f414f1","line":18,"updated":"2015-07-09 06:30:56.000000000","message":"BP or bug link?","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"3d27771d956ad92e5dce3e2e73b980f03de1504b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"management of volumes that are not currently mapped to a server on the"},{"line_number":16,"context_line":"Storage Center. id-types of source-id (DeviceId on the Storage Center)"},{"line_number":17,"context_line":"and source-name (Volume Name on the Storage Center) are supported."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Iecc93ce1654111e1292cfc1931d1eda4abd08595"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba3cc151_9f6ae8b1","line":18,"in_reply_to":"ba3cc151_88f414f1","updated":"2015-07-09 16:05:24.000000000","message":"I opened neither mcginnis keeps claiming it isn\u0027t required.  Is it?","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"}],"cinder/tests/unit/test_dellscapi.py":[{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"ed264dca20862aa63eca6df4697ac94b3e8cca62","unresolved":false,"context_lines":[{"line_number":4801,"context_line":"                                           mock_open_connection,"},{"line_number":4802,"context_line":"                                           mock_init):"},{"line_number":4803,"context_line":""},{"line_number":4804,"context_line":"        # Same as above only we don\u0027t have a volume folder."},{"line_number":4805,"context_line":"        newname \u003d \u0027guid\u0027"},{"line_number":4806,"context_line":"        existing \u003d {\u0027source-name\u0027: \u0027scvolname\u0027}"},{"line_number":4807,"context_line":"        self.assertRaises(exception.ManageExistingInvalidReference,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_3899b279","line":4804,"updated":"2015-07-08 18:34:09.000000000","message":"Is this comment correct.  Previous test has same comment","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"ed264dca20862aa63eca6df4697ac94b3e8cca62","unresolved":false,"context_lines":[{"line_number":4822,"context_line":"                                                mock_open_connection,"},{"line_number":4823,"context_line":"                                                mock_init):"},{"line_number":4824,"context_line":""},{"line_number":4825,"context_line":"        # Same as above only we don\u0027t have a volume folder."},{"line_number":4826,"context_line":"        newname \u003d \u0027guid\u0027"},{"line_number":4827,"context_line":"        existing \u003d {\u0027source-name\u0027: \u0027scvolname\u0027}"},{"line_number":4828,"context_line":"        self.assertRaises(exception.ManageExistingInvalidReference,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_d888c624","line":4825,"updated":"2015-07-08 18:34:09.000000000","message":"Is this comment correct.  Previous test has same comment","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"ed264dca20862aa63eca6df4697ac94b3e8cca62","unresolved":false,"context_lines":[{"line_number":4848,"context_line":"                                      mock_open_connection,"},{"line_number":4849,"context_line":"                                      mock_init):"},{"line_number":4850,"context_line":""},{"line_number":4851,"context_line":"        # Same as above only we don\u0027t have a volume folder."},{"line_number":4852,"context_line":"        newname \u003d \u0027guid\u0027"},{"line_number":4853,"context_line":"        existing \u003d {\u0027source-name\u0027: \u0027scvolname\u0027}"},{"line_number":4854,"context_line":"        self.assertRaises(exception.VolumeBackendAPIException,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_987ebe19","line":4851,"updated":"2015-07-08 18:34:09.000000000","message":"Is this comment correct.  Previous test has same comment","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"ed264dca20862aa63eca6df4697ac94b3e8cca62","unresolved":false,"context_lines":[{"line_number":5052,"context_line":"                           mock_close_connection,"},{"line_number":5053,"context_line":"                           mock_open_connection,"},{"line_number":5054,"context_line":"                           mock_init):"},{"line_number":5055,"context_line":"        # Same as above only we don\u0027t have a volume folder."},{"line_number":5056,"context_line":"        scvolume \u003d {\u0027name\u0027: \u0027guid\u0027}"},{"line_number":5057,"context_line":"        expected_url \u003d \u0027StorageCenter/ScVolume/100\u0027"},{"line_number":5058,"context_line":"        newname \u003d \u0027Unmanaged_\u0027 + scvolume[\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_3870d23f","line":5055,"updated":"2015-07-08 18:34:09.000000000","message":"Is this comment correct.  Previous test has same comment","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"}],"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"ed264dca20862aa63eca6df4697ac94b3e8cca62","unresolved":false,"context_lines":[{"line_number":622,"context_line":"            if name is not None:"},{"line_number":623,"context_line":"                pf.append(\u0027Name\u0027, name)"},{"line_number":624,"context_line":"            if deviceid is not None:"},{"line_number":625,"context_line":"                pf.append(\u0027DeviceId\u0027, deviceid)"},{"line_number":626,"context_line":"            # set folderPath"},{"line_number":627,"context_line":"            if filterbyvfname:"},{"line_number":628,"context_line":"                vfname \u003d (self.vfname if self.vfname.endswith(\u0027/\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_d81206b4","line":625,"updated":"2015-07-08 18:34:09.000000000","message":"If name and deviceid are both None, should method just return None right away?","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"30e6bd698e5caef7e50e001d9065b094f1c0de50","unresolved":false,"context_lines":[{"line_number":622,"context_line":"            if name is not None:"},{"line_number":623,"context_line":"                pf.append(\u0027Name\u0027, name)"},{"line_number":624,"context_line":"            if deviceid is not None:"},{"line_number":625,"context_line":"                pf.append(\u0027DeviceId\u0027, deviceid)"},{"line_number":626,"context_line":"            # set folderPath"},{"line_number":627,"context_line":"            if filterbyvfname:"},{"line_number":628,"context_line":"                vfname \u003d (self.vfname if self.vfname.endswith(\u0027/\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_3e051b79","line":625,"in_reply_to":"ba3cc151_d81206b4","updated":"2015-07-08 18:43:54.000000000","message":"It does.  If name or deviceid it runs this code else it retuns result which is None.","commit_id":"a300ebf0db8b02c13bdf0ad65bc38f57d2bc976d"}]}