)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"43356e8b9addb1119e54409cd5f22e2312ef459f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Yucong Feng \u003cyfeng@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-02-19 23:34:40 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Return all target_wwpns for FC storwize."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Storwize FC terminate_connection was using \u0027lsfabric\u0027 to lookup"},{"line_number":10,"context_line":"target_wwpns active on the fabric to be used to terminate"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bae84128_d62aa48f","line":7,"range":{"start_line":7,"start_character":39,"end_line":7,"end_character":40},"updated":"2016-02-22 20:24:04.000000000","message":"First line of commit message should not end with a period. \n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"67c9616ab42ae144031a57f92f8ee97b28ee2a7c"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"43356e8b9addb1119e54409cd5f22e2312ef459f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Storwize FC terminate_connection was using \u0027lsfabric\u0027 to lookup"},{"line_number":10,"context_line":"target_wwpns active on the fabric to be used to terminate"},{"line_number":11,"context_line":"the connection. However during a delete of a NPIV VM, it is no longer active,"},{"line_number":12,"context_line":"it is not logged onto the fabric. The target list will return empty."},{"line_number":13,"context_line":"Fix is to return all the available wwpns from \u0027storage_nodes\u0027."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bae84128_d653c437","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":77},"updated":"2016-02-22 20:24:04.000000000","message":"Lines should be wrapped at 72 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"67c9616ab42ae144031a57f92f8ee97b28ee2a7c"}],"cinder/tests/unit/test_storwize_svc.py":[{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"e5b1043803816fd34cf96e398e5d9f6e426ce42f","unresolved":false,"context_lines":[{"line_number":2315,"context_line":"                              }"},{"line_number":2316,"context_line":"                     }"},{"line_number":2317,"context_line":""},{"line_number":2318,"context_line":"        self.assertEqual(term_data, term_ret)"},{"line_number":2319,"context_line":""},{"line_number":2320,"context_line":"    def test_storwize_svc_fc_host_maps(self):"},{"line_number":2321,"context_line":"        # Create two volumes to be used in mappings"}],"source_content_type":"text/x-python","patch_set":4,"id":"bae84128_dca9907d","line":2318,"range":{"start_line":2318,"start_character":13,"end_line":2318,"end_character":24},"updated":"2016-02-25 01:53:58.000000000","message":"assertEqual for Dict/list is not compatible in python 2.7 and 3.4. You may try to split the assertEqual into small pieces or something else.","commit_id":"c9e733044ca4e3019d6d7112e2514da24bea074d"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_fc.py":[{"author":{"_account_id":14532,"name":"Gerald McBrearty","email":"gfm@us.ibm.com","username":"gfm"},"change_message_id":"52b0c13aac9cc358c8afd34babe979e3a5482b8d","unresolved":false,"context_lines":[{"line_number":291,"context_line":"                             \"target map.\"))"},{"line_number":292,"context_line":"                # Build info data structure for zone removing"},{"line_number":293,"context_line":"                if \u0027wwpns\u0027 in connector and host_name:"},{"line_number":294,"context_line":"                    target_wwpns \u003d []"},{"line_number":295,"context_line":"                    # Returning all target_wwpns in storage_nodes, since"},{"line_number":296,"context_line":"                    # we cannot determine which wwpns are logged in during"},{"line_number":297,"context_line":"                    # a VM deletion."}],"source_content_type":"text/x-python","patch_set":5,"id":"9aed3d3a_0181e14a","line":294,"updated":"2016-02-25 19:47:53.000000000","message":"I agree with Jay we should have left the npiv switch.\nIf the VM is shutdown such as the case when a VM is being deleted all of the ports would be logged out. In power the vfc adapter is a pair of ports with one only being active until the vm is migrated. With lsfabric we would end up deleting half of the zones instead of all of them.\nBut this is terminate connection, the zone manager would end up deleting only the zones it finds anyway.","commit_id":"f8636888590ddb3339eafad9c00e4806df486238"}]}