)]}'
{"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"096b7406c29c30c2f57c0668e5404ef24f3fb5fc","unresolved":false,"context_lines":[{"line_number":761,"context_line":"            # the mapping we have been configured to use."},{"line_number":762,"context_line":"            for mapping in mappings:"},{"line_number":763,"context_line":"                # Find the controller port for this mapping"},{"line_number":764,"context_line":"                cport \u003d mapping.get(\u0027controllerPort\u0027)"},{"line_number":765,"context_line":"                controllerport \u003d self._find_controller_port("},{"line_number":766,"context_line":"                    self._get_id(cport))"},{"line_number":767,"context_line":"                if controllerport is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_bb2bf692","line":764,"updated":"2015-03-24 21:56:37.000000000","message":"Do we have a build in assumption of a single port per mapping?","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"8fd5ac07e55929fc7cd0f9c136bb5092ac72bb0a","unresolved":false,"context_lines":[{"line_number":761,"context_line":"            # the mapping we have been configured to use."},{"line_number":762,"context_line":"            for mapping in mappings:"},{"line_number":763,"context_line":"                # Find the controller port for this mapping"},{"line_number":764,"context_line":"                cport \u003d mapping.get(\u0027controllerPort\u0027)"},{"line_number":765,"context_line":"                controllerport \u003d self._find_controller_port("},{"line_number":766,"context_line":"                    self._get_id(cport))"},{"line_number":767,"context_line":"                if controllerport is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_ee8cce37","line":764,"in_reply_to":"1a6ced46_bb2bf692","updated":"2015-03-24 22:13:41.000000000","message":"Per mapping entry, yes.","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"68d6259099c34fe440b9d623f96628a72da93536","unresolved":false,"context_lines":[{"line_number":770,"context_line":"                    wwn \u003d controllerport.get(\u0027wwn\u0027,"},{"line_number":771,"context_line":"                                             controllerport.get(\u0027WWN\u0027))"},{"line_number":772,"context_line":"                    if wwn is None:"},{"line_number":773,"context_line":"                        LOG.error(_LE(\u0027Find_wwns: Unable to find port wwn\u0027))"},{"line_number":774,"context_line":"                    serverhba \u003d mapping.get(\u0027serverHba\u0027)"},{"line_number":775,"context_line":"                    if wwn is not None and serverhba is not None:"},{"line_number":776,"context_line":"                        hbaname \u003d serverhba.get(\u0027instanceName\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_a4fe7c36","line":773,"updated":"2015-03-24 20:43:08.000000000","message":"Do you want to log which mapping and controllerport faild to have a wwn?","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"37924e08bcbf7bd34e87c0712359fa6c46c13935","unresolved":false,"context_lines":[{"line_number":770,"context_line":"                    wwn \u003d controllerport.get(\u0027wwn\u0027,"},{"line_number":771,"context_line":"                                             controllerport.get(\u0027WWN\u0027))"},{"line_number":772,"context_line":"                    if wwn is None:"},{"line_number":773,"context_line":"                        LOG.error(_LE(\u0027Find_wwns: Unable to find port wwn\u0027))"},{"line_number":774,"context_line":"                    serverhba \u003d mapping.get(\u0027serverHba\u0027)"},{"line_number":775,"context_line":"                    if wwn is not None and serverhba is not None:"},{"line_number":776,"context_line":"                        hbaname \u003d serverhba.get(\u0027instanceName\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_f83bb499","line":773,"in_reply_to":"1a6ced46_a4fe7c36","updated":"2015-03-24 21:05:49.000000000","message":"It isn\u0027t meaningful.  It really has to be there.  Most likely this is a REST API change (per the original bug) or a spurious REST API failure.\n\nIt does beg the question of whether I should try to struggle on at all.","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"68d6259099c34fe440b9d623f96628a72da93536","unresolved":false,"context_lines":[{"line_number":772,"context_line":"                    if wwn is None:"},{"line_number":773,"context_line":"                        LOG.error(_LE(\u0027Find_wwns: Unable to find port wwn\u0027))"},{"line_number":774,"context_line":"                    serverhba \u003d mapping.get(\u0027serverHba\u0027)"},{"line_number":775,"context_line":"                    if wwn is not None and serverhba is not None:"},{"line_number":776,"context_line":"                        hbaname \u003d serverhba.get(\u0027instanceName\u0027)"},{"line_number":777,"context_line":"                        if hbaname in initiators:"},{"line_number":778,"context_line":"                            if itmap.get(hbaname) is None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_c49458e4","line":775,"updated":"2015-03-24 20:43:08.000000000","message":"Is it noteworth enough that serverhba would be None that you\u0027d want to log it?","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"37924e08bcbf7bd34e87c0712359fa6c46c13935","unresolved":false,"context_lines":[{"line_number":772,"context_line":"                    if wwn is None:"},{"line_number":773,"context_line":"                        LOG.error(_LE(\u0027Find_wwns: Unable to find port wwn\u0027))"},{"line_number":774,"context_line":"                    serverhba \u003d mapping.get(\u0027serverHba\u0027)"},{"line_number":775,"context_line":"                    if wwn is not None and serverhba is not None:"},{"line_number":776,"context_line":"                        hbaname \u003d serverhba.get(\u0027instanceName\u0027)"},{"line_number":777,"context_line":"                        if hbaname in initiators:"},{"line_number":778,"context_line":"                            if itmap.get(hbaname) is None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_58e848e2","line":775,"in_reply_to":"1a6ced46_c49458e4","updated":"2015-03-24 21:05:49.000000000","message":"I\u0027m not sure it would add enough value to add a log string to Cinder at this point.","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"68d6259099c34fe440b9d623f96628a72da93536","unresolved":false,"context_lines":[{"line_number":785,"context_line":"                                lun \u003d mappinglun"},{"line_number":786,"context_line":"                            elif lun !\u003d mappinglun:"},{"line_number":787,"context_line":"                                LOG.warning(_LW(\u0027Inconsistent Luns.\u0027))"},{"line_number":788,"context_line":"        else:"},{"line_number":789,"context_line":"            LOG.error(_LE(\u0027Find_wwns: Volume appears unmapped\u0027))"},{"line_number":790,"context_line":"        LOG.debug(lun)"},{"line_number":791,"context_line":"        LOG.debug(wwns)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_0465901d","line":788,"updated":"2015-03-24 20:43:08.000000000","message":"Log which volume?","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"37924e08bcbf7bd34e87c0712359fa6c46c13935","unresolved":false,"context_lines":[{"line_number":785,"context_line":"                                lun \u003d mappinglun"},{"line_number":786,"context_line":"                            elif lun !\u003d mappinglun:"},{"line_number":787,"context_line":"                                LOG.warning(_LW(\u0027Inconsistent Luns.\u0027))"},{"line_number":788,"context_line":"        else:"},{"line_number":789,"context_line":"            LOG.error(_LE(\u0027Find_wwns: Volume appears unmapped\u0027))"},{"line_number":790,"context_line":"        LOG.debug(lun)"},{"line_number":791,"context_line":"        LOG.debug(wwns)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_584cc842","line":788,"in_reply_to":"1a6ced46_0465901d","updated":"2015-03-24 21:05:49.000000000","message":"Logged elsewhere.","commit_id":"cbc9fe5d61c6ea4aa3ca0fd97b9ccebd4bb2fe30"}]}