)]}'
{"cinder/volume/drivers/kaminario/kaminario_fc.py":[{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"0ae41769ef99c1aa81357b96b7cd45ccc0ef87d5","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        target_wwpns \u003d self.get_target_info()"},{"line_number":48,"context_line":"        # Map volume."},{"line_number":49,"context_line":"        lun \u003d self.k2_initialize_connection(volume, connector)"},{"line_number":50,"context_line":"        # Create initiator-target mapping."},{"line_number":51,"context_line":"        initiator_target_map \u003d {}"},{"line_number":52,"context_line":"        for wwpn in connector[\u0027wwpns\u0027]:"},{"line_number":53,"context_line":"            initiator_target_map[wwpn] \u003d target_wwpns"},{"line_number":54,"context_line":"        # Return target volume information."},{"line_number":55,"context_line":"        return {\u0027driver_volume_type\u0027: \u0027fibre_channel\u0027,"},{"line_number":56,"context_line":"                \u0027data\u0027: {\"target_discovered\": True,"}],"source_content_type":"text/x-python","patch_set":7,"id":"3aaa91ec_ac4fe668","line":53,"range":{"start_line":50,"start_character":8,"end_line":53,"end_character":53},"updated":"2016-06-28 19:34:50.000000000","message":"Unfortunately it isn\u0027t so simple :( There is a whole song and dance that should be performed....\n\nIIRC you need to use the FCZM to see what is already zoned and just ask for new zones that need to be setup. Check out some of the other FC drivers, you will see something like https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/pure.py#L1760  or https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/hpe/hpe_3par_fc.py#L347\n\nAt some point we should really simplify this and have shared/generic code, but it is what it is.","commit_id":"c4517d04f2b19851b9f00f0669ae802dcfddccb9"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"0ae41769ef99c1aa81357b96b7cd45ccc0ef87d5","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                         \"target_wwn\": target_wwpns,"},{"line_number":59,"context_line":"                         \"initiator_target_map\": initiator_target_map}}"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    @fczm_utils.RemoveFCZone"},{"line_number":62,"context_line":"    def terminate_connection(self, volume, connector, **kwargs):"},{"line_number":63,"context_line":"        super(KaminarioFCDriver, self).terminate_connection(volume, connector)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    @kaminario_logger"},{"line_number":66,"context_line":"    def get_target_info(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"3aaa91ec_2cd8167f","line":63,"range":{"start_line":61,"start_character":4,"end_line":63,"end_character":78},"updated":"2016-06-28 19:34:50.000000000","message":"There is some FCZM fun here too thats missing, you need to decide when to remove the zone, again check out https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/hpe/hpe_3par_fc.py#L321-L329 or https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/pure.py#L1795","commit_id":"c4517d04f2b19851b9f00f0669ae802dcfddccb9"}]}