)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10439,"name":"Xi Yang","email":"yang.xi@99cloud.net","username":"phenom"},"change_message_id":"f04ef746f6ebdea5b0b9391d8ecd21e2e6eacb68","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Helen Walsh \u003chelen.walsh@emc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-26 12:22:19 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - Replacing depreciated API EMCGetTargetEndpoints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"8.x version of SE no longer supports EMCGetTargetEndpoints"},{"line_number":10,"context_line":"API. Replacing with an association to EMC_FCSCSIProtocolEndpoint."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9a8ffd7b_2b8b7020","line":7,"updated":"2015-11-30 05:41:41.000000000","message":"Seems typo of \u0027deprecated\u0027 here.","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52330a1e2e04048ec8d345d246568fa876652d9c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Helen Walsh \u003chelen.walsh@emc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-26 12:22:19 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - Replacing depreciated API EMCGetTargetEndpoints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"8.x version of SE no longer supports EMCGetTargetEndpoints"},{"line_number":10,"context_line":"API. Replacing with an association to EMC_FCSCSIProtocolEndpoint."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9a8ffd7b_dd65b81d","line":7,"in_reply_to":"9a8ffd7b_2b8b7020","updated":"2015-11-30 12:45:46.000000000","message":"I will change this.  Thanks","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":10439,"name":"Xi Yang","email":"yang.xi@99cloud.net","username":"phenom"},"change_message_id":"f04ef746f6ebdea5b0b9391d8ecd21e2e6eacb68","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - Replacing depreciated API EMCGetTargetEndpoints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"8.x version of SE no longer supports EMCGetTargetEndpoints"},{"line_number":10,"context_line":"API. Replacing with an association to EMC_FCSCSIProtocolEndpoint."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1512791"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9a8ffd7b_eb8008f9","line":9,"updated":"2015-11-30 05:41:41.000000000","message":"Does SE means \u0027Solution Enabler\u0027? Better to expand it.","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52330a1e2e04048ec8d345d246568fa876652d9c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - Replacing depreciated API EMCGetTargetEndpoints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"8.x version of SE no longer supports EMCGetTargetEndpoints"},{"line_number":10,"context_line":"API. Replacing with an association to EMC_FCSCSIProtocolEndpoint."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1512791"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9a8ffd7b_3df6445d","line":9,"in_reply_to":"9a8ffd7b_eb8008f9","updated":"2015-11-30 12:45:46.000000000","message":"I will","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"f1de8a5cddfee1fef20bec97b937353fb492e3df","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Helen Walsh \u003chelen.walsh@emc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-12-20 23:47:05 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - Replacing deprecated API EMCGetTargetEndpoints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"8.x version of Solution Enabler no longer supports"},{"line_number":10,"context_line":"EMCGetTargetEndpoints API. Replacing with an association"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"fa69d971_80dc0374","line":7,"range":{"start_line":7,"start_character":36,"end_line":7,"end_character":57},"updated":"2016-01-05 17:39:53.000000000","message":"Summary line shouldn\u0027t go over 50 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"caa7c828bbef0118ecbceeab2f397d4f7f845c8e"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"f1de8a5cddfee1fef20bec97b937353fb492e3df","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"8.x version of Solution Enabler no longer supports"},{"line_number":10,"context_line":"EMCGetTargetEndpoints API. Replacing with an association"},{"line_number":11,"context_line":"to EMC_FCSCSIProtocolEndpoint."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1512791"},{"line_number":14,"context_line":"Change-Id: I49b49c3deb372cbbf77e87313d0cbc155cdb8984"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"fa69d971_b69b9165","line":11,"range":{"start_line":11,"start_character":29,"end_line":11,"end_character":30},"updated":"2016-01-05 17:39:53.000000000","message":"Can you also talk a little bit about the methods that were added here? Would be helpful for down the road if people need to know when they were implemented.","commit_id":"caa7c828bbef0118ecbceeab2f397d4f7f845c8e"}],"cinder/tests/unit/test_emc_vmax.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"cb051db1711cb7e7457f92a95c1670a39bfda5b5","unresolved":false,"context_lines":[{"line_number":6860,"context_line":"        configuration.config_group \u003d \u0027UtilsTests\u0027"},{"line_number":6861,"context_line":""},{"line_number":6862,"context_line":"        self.stubs.Set(emc_vmax_common.EMCVMAXCommon, \u0027_get_ecom_connection\u0027,"},{"line_number":6863,"context_line":"                       self.fake_ecom_connection)"},{"line_number":6864,"context_line":"        self.stubs.Set(emc_vmax_common.EMCVMAXCommon, \u0027_gather_info\u0027,"},{"line_number":6865,"context_line":"                       self.fake_gather_info)"},{"line_number":6866,"context_line":"        instancename \u003d FakeCIMInstanceName()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_bdb23d04","line":6863,"updated":"2015-11-12 16:22:10.000000000","message":"Please, use mock instead of mox for new tests","commit_id":"b9fef645e2c7059ed40a96fa241738d5debb8330"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"f93d70a3ae49d07d2f15175b0af0293b59534f34","unresolved":false,"context_lines":[{"line_number":6860,"context_line":"        configuration.config_group \u003d \u0027UtilsTests\u0027"},{"line_number":6861,"context_line":""},{"line_number":6862,"context_line":"        self.stubs.Set(emc_vmax_common.EMCVMAXCommon, \u0027_get_ecom_connection\u0027,"},{"line_number":6863,"context_line":"                       self.fake_ecom_connection)"},{"line_number":6864,"context_line":"        self.stubs.Set(emc_vmax_common.EMCVMAXCommon, \u0027_gather_info\u0027,"},{"line_number":6865,"context_line":"                       self.fake_gather_info)"},{"line_number":6866,"context_line":"        instancename \u003d FakeCIMInstanceName()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_fa50bd0c","line":6863,"in_reply_to":"da85f559_bdb23d04","updated":"2015-11-12 20:24:54.000000000","message":"Ok. Thanks","commit_id":"b9fef645e2c7059ed40a96fa241738d5debb8330"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"0bb18bc2071b485e3b69dc76c8e8094c105fe857","unresolved":false,"context_lines":[{"line_number":6907,"context_line":"            exception.VolumeBackendAPIException,"},{"line_number":6908,"context_line":"            common.utils.get_target_endpoints,"},{"line_number":6909,"context_line":"            conn, hardwareId)"},{"line_number":6910,"context_line":""},{"line_number":6911,"context_line":"    def _cleanup(self, tempdir, config_file_path):"},{"line_number":6912,"context_line":"        bExists \u003d os.path.exists(config_file_path)"},{"line_number":6913,"context_line":"        if bExists:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_d7b5a2ec","line":6910,"updated":"2015-11-12 16:02:23.000000000","message":"I think we need to add unit tests for covering the branches in get_protocol_controller in file emc_vmax_utils.py.","commit_id":"b9fef645e2c7059ed40a96fa241738d5debb8330"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"f93d70a3ae49d07d2f15175b0af0293b59534f34","unresolved":false,"context_lines":[{"line_number":6907,"context_line":"            exception.VolumeBackendAPIException,"},{"line_number":6908,"context_line":"            common.utils.get_target_endpoints,"},{"line_number":6909,"context_line":"            conn, hardwareId)"},{"line_number":6910,"context_line":""},{"line_number":6911,"context_line":"    def _cleanup(self, tempdir, config_file_path):"},{"line_number":6912,"context_line":"        bExists \u003d os.path.exists(config_file_path)"},{"line_number":6913,"context_line":"        if bExists:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_927ac822","line":6910,"in_reply_to":"da85f559_d7b5a2ec","updated":"2015-11-12 20:24:54.000000000","message":"Thanks Jacob, we can get that done","commit_id":"b9fef645e2c7059ed40a96fa241738d5debb8330"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"d567043a9403ad45bbfb365fb7672d1d6ccecc9d","unresolved":false,"context_lines":[{"line_number":6866,"context_line":"        self.driver \u003d driver"},{"line_number":6867,"context_line":"        self.driver.utils \u003d emc_vmax_utils.EMCVMAXUtils(object)"},{"line_number":6868,"context_line":""},{"line_number":6869,"context_line":"    def fake_ecom_connection(self):"},{"line_number":6870,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":6871,"context_line":"        return conn"},{"line_number":6872,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_8df0d068","line":6869,"updated":"2015-11-26 13:57:16.000000000","message":"This method is not useful and should be removed","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52330a1e2e04048ec8d345d246568fa876652d9c","unresolved":false,"context_lines":[{"line_number":6866,"context_line":"        self.driver \u003d driver"},{"line_number":6867,"context_line":"        self.driver.utils \u003d emc_vmax_utils.EMCVMAXUtils(object)"},{"line_number":6868,"context_line":""},{"line_number":6869,"context_line":"    def fake_ecom_connection(self):"},{"line_number":6870,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":6871,"context_line":"        return conn"},{"line_number":6872,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_98fc1e73","line":6869,"in_reply_to":"9a8ffd7b_8df0d068","updated":"2015-11-30 12:45:46.000000000","message":"Will do.  Thanks","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"d567043a9403ad45bbfb365fb7672d1d6ccecc9d","unresolved":false,"context_lines":[{"line_number":6870,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":6871,"context_line":"        return conn"},{"line_number":6872,"context_line":""},{"line_number":6873,"context_line":"    def fake_gather_info(self):"},{"line_number":6874,"context_line":"        return"},{"line_number":6875,"context_line":""},{"line_number":6876,"context_line":"    @mock.patch.object("}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_8d9770eb","line":6873,"updated":"2015-11-26 13:57:16.000000000","message":"The same as above.","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52330a1e2e04048ec8d345d246568fa876652d9c","unresolved":false,"context_lines":[{"line_number":6870,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":6871,"context_line":"        return conn"},{"line_number":6872,"context_line":""},{"line_number":6873,"context_line":"    def fake_gather_info(self):"},{"line_number":6874,"context_line":"        return"},{"line_number":6875,"context_line":""},{"line_number":6876,"context_line":"    @mock.patch.object("}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_3843b2b0","line":6873,"in_reply_to":"9a8ffd7b_8d9770eb","updated":"2015-11-30 12:45:46.000000000","message":"Will do","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"f0e795afc2559bf61353525f67cd18cc24e3fbfd","unresolved":false,"context_lines":[{"line_number":7449,"context_line":"        driver.db \u003d FakeDB()"},{"line_number":7450,"context_line":"        self.driver \u003d driver"},{"line_number":7451,"context_line":"        self.driver.utils \u003d emc_vmax_utils.EMCVMAXUtils(object)"},{"line_number":7452,"context_line":""},{"line_number":7453,"context_line":"    def fake_ecom_connection(self):"},{"line_number":7454,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":7455,"context_line":"        return conn"}],"source_content_type":"text/x-python","patch_set":18,"id":"5a5ae5dd_37eca98e","line":7452,"updated":"2016-02-04 23:44:42.000000000","message":"We don\u0027t seem to be using this function (or the one after it)","commit_id":"88531f6f10a753a3fcdae80505bbe3e74b18b534"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"2a040447ac812c8af68d8e5c0484ee90a48fa218","unresolved":false,"context_lines":[{"line_number":7449,"context_line":"        driver.db \u003d FakeDB()"},{"line_number":7450,"context_line":"        self.driver \u003d driver"},{"line_number":7451,"context_line":"        self.driver.utils \u003d emc_vmax_utils.EMCVMAXUtils(object)"},{"line_number":7452,"context_line":""},{"line_number":7453,"context_line":"    def fake_ecom_connection(self):"},{"line_number":7454,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":7455,"context_line":"        return conn"}],"source_content_type":"text/x-python","patch_set":18,"id":"5a5ae5dd_d4bdb702","line":7452,"in_reply_to":"5a5ae5dd_37eca98e","updated":"2016-02-08 10:17:42.000000000","message":"Thanks Ryan.  I will clean up where applicable.","commit_id":"88531f6f10a753a3fcdae80505bbe3e74b18b534"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"f0e795afc2559bf61353525f67cd18cc24e3fbfd","unresolved":false,"context_lines":[{"line_number":7471,"context_line":""},{"line_number":7472,"context_line":"    def test_get_protocol_controller(self):"},{"line_number":7473,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":7474,"context_line":"        hardwareid \u003d 123456789012345"},{"line_number":7475,"context_line":"        result \u003d self.driver.utils.get_protocol_controller(conn, hardwareid)"},{"line_number":7476,"context_line":"        self.assertEqual("},{"line_number":7477,"context_line":"            ({\u0027CreationClassName\u0027: \u0027Symm_LunMaskingView\u0027,"}],"source_content_type":"text/x-python","patch_set":18,"id":"5a5ae5dd_f7acd136","line":7474,"updated":"2016-02-04 23:44:42.000000000","message":"I\u0027m not following why we ned to add the mock.patch on line 7460, if the default return value of emc_vmax_utils.EMCVMAXUtils.get_protocol_controller() seems to be {\u0027CreationClassName\u0027: \u0027Symm_LunMaskingView\u0027,\n              \u0027ElementName\u0027: \u0027myMaskingView\u0027}\n??","commit_id":"88531f6f10a753a3fcdae80505bbe3e74b18b534"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"2a040447ac812c8af68d8e5c0484ee90a48fa218","unresolved":false,"context_lines":[{"line_number":7471,"context_line":""},{"line_number":7472,"context_line":"    def test_get_protocol_controller(self):"},{"line_number":7473,"context_line":"        conn \u003d FakeEcomConnection()"},{"line_number":7474,"context_line":"        hardwareid \u003d 123456789012345"},{"line_number":7475,"context_line":"        result \u003d self.driver.utils.get_protocol_controller(conn, hardwareid)"},{"line_number":7476,"context_line":"        self.assertEqual("},{"line_number":7477,"context_line":"            ({\u0027CreationClassName\u0027: \u0027Symm_LunMaskingView\u0027,"}],"source_content_type":"text/x-python","patch_set":18,"id":"5a5ae5dd_f44f9331","line":7474,"in_reply_to":"5a5ae5dd_f7acd136","updated":"2016-02-08 10:17:42.000000000","message":"I will look into this.  Thank you.","commit_id":"88531f6f10a753a3fcdae80505bbe3e74b18b534"}],"cinder/volume/drivers/emc/emc_vmax_utils.py":[{"author":{"_account_id":10439,"name":"Xi Yang","email":"yang.xi@99cloud.net","username":"phenom"},"change_message_id":"f04ef746f6ebdea5b0b9391d8ecd21e2e6eacb68","unresolved":false,"context_lines":[{"line_number":2434,"context_line":""},{"line_number":2435,"context_line":"        :param conn: the connection to the ecom server"},{"line_number":2436,"context_line":"        :param hardwareId: the hardware Id"},{"line_number":2437,"context_line":"        :returns: int -- return code"},{"line_number":2438,"context_line":"        :returns: targetEndpoints"},{"line_number":2439,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2440,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_8ba7fca5","line":2437,"updated":"2015-11-30 05:41:41.000000000","message":"Does this function return int?","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52330a1e2e04048ec8d345d246568fa876652d9c","unresolved":false,"context_lines":[{"line_number":2434,"context_line":""},{"line_number":2435,"context_line":"        :param conn: the connection to the ecom server"},{"line_number":2436,"context_line":"        :param hardwareId: the hardware Id"},{"line_number":2437,"context_line":"        :returns: int -- return code"},{"line_number":2438,"context_line":"        :returns: targetEndpoints"},{"line_number":2439,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2440,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_3d008401","line":2437,"in_reply_to":"9a8ffd7b_8ba7fca5","updated":"2015-11-30 12:45:46.000000000","message":"No, I does not, good catch","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":10439,"name":"Xi Yang","email":"yang.xi@99cloud.net","username":"phenom"},"change_message_id":"f04ef746f6ebdea5b0b9391d8ecd21e2e6eacb68","unresolved":false,"context_lines":[{"line_number":2454,"context_line":""},{"line_number":2455,"context_line":"        :param conn: the ecom connection"},{"line_number":2456,"context_line":"        :param hardwareinstancename: the hardware instance name"},{"line_number":2457,"context_line":"        :returns: protocolControllerInstanceName"},{"line_number":2458,"context_line":"        \"\"\""},{"line_number":2459,"context_line":"        protocolControllerInstanceName \u003d None"},{"line_number":2460,"context_line":"        protocol_controllers \u003d conn.AssociatorNames("}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_6bc3b838","line":2457,"updated":"2015-11-30 05:41:41.000000000","message":"Better to add \u0027raises\u0027 here as you did in the above function","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52330a1e2e04048ec8d345d246568fa876652d9c","unresolved":false,"context_lines":[{"line_number":2454,"context_line":""},{"line_number":2455,"context_line":"        :param conn: the ecom connection"},{"line_number":2456,"context_line":"        :param hardwareinstancename: the hardware instance name"},{"line_number":2457,"context_line":"        :returns: protocolControllerInstanceName"},{"line_number":2458,"context_line":"        \"\"\""},{"line_number":2459,"context_line":"        protocolControllerInstanceName \u003d None"},{"line_number":2460,"context_line":"        protocol_controllers \u003d conn.AssociatorNames("}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_78113a51","line":2457,"in_reply_to":"9a8ffd7b_6bc3b838","updated":"2015-11-30 12:45:46.000000000","message":"I will","commit_id":"4511072ca46bc8197e3b422509264931bf516494"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"f0e795afc2559bf61353525f67cd18cc24e3fbfd","unresolved":false,"context_lines":[{"line_number":2471,"context_line":"        :returns: targetEndpoints"},{"line_number":2472,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2473,"context_line":"        \"\"\""},{"line_number":2474,"context_line":"        targetEndpoints \u003d None"},{"line_number":2475,"context_line":"        protocolControllerInstanceName \u003d self.get_protocol_controller("},{"line_number":2476,"context_line":"            conn, hardwareId)"},{"line_number":2477,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"5a5ae5dd_d75f553f","line":2474,"updated":"2016-02-04 23:44:42.000000000","message":"Don\u0027t need to define targetEndpoints here since we\u0027re overwriting always on line 2478","commit_id":"88531f6f10a753a3fcdae80505bbe3e74b18b534"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"2a040447ac812c8af68d8e5c0484ee90a48fa218","unresolved":false,"context_lines":[{"line_number":2471,"context_line":"        :returns: targetEndpoints"},{"line_number":2472,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2473,"context_line":"        \"\"\""},{"line_number":2474,"context_line":"        targetEndpoints \u003d None"},{"line_number":2475,"context_line":"        protocolControllerInstanceName \u003d self.get_protocol_controller("},{"line_number":2476,"context_line":"            conn, hardwareId)"},{"line_number":2477,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"5a5ae5dd_14fd8fb3","line":2474,"in_reply_to":"5a5ae5dd_d75f553f","updated":"2016-02-08 10:17:42.000000000","message":"I will remove it.  Thank you","commit_id":"88531f6f10a753a3fcdae80505bbe3e74b18b534"}]}