)]}'
{"ironic/common/molds.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"64575ef271aff6729c66cefb0a0f96b2ad528a32","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    auth_header \u003d _get_auth_header(task)"},{"line_number":82,"context_line":"    response \u003d _request(url, auth_header)"},{"line_number":83,"context_line":"    if response.status_code \u003d\u003d requests.codes.ok:"},{"line_number":84,"context_line":"        if len(response.content) \u003d\u003d 0:"},{"line_number":85,"context_line":"            raise exception.IronicException(_("},{"line_number":86,"context_line":"                \"Configuration mold for node %s at %s is empty\")"},{"line_number":87,"context_line":"                % (task.node.uuid, url))"}],"source_content_type":"text/x-python","patch_set":1,"id":"0527e0ed_b097e984","line":84,"updated":"2021-08-20 11:57:54.000000000","message":"better:\n\n if not response.content:","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"1e11b0a39971659ac091657791465a633d2a4135","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    auth_header \u003d _get_auth_header(task)"},{"line_number":82,"context_line":"    response \u003d _request(url, auth_header)"},{"line_number":83,"context_line":"    if response.status_code \u003d\u003d requests.codes.ok:"},{"line_number":84,"context_line":"        if len(response.content) \u003d\u003d 0:"},{"line_number":85,"context_line":"            raise exception.IronicException(_("},{"line_number":86,"context_line":"                \"Configuration mold for node %s at %s is empty\")"},{"line_number":87,"context_line":"                % (task.node.uuid, url))"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4cfd68_e54c0a7d","line":84,"in_reply_to":"0527e0ed_b097e984","updated":"2021-08-23 10:14:05.000000000","message":"done","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"64575ef271aff6729c66cefb0a0f96b2ad528a32","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    if response.status_code \u003d\u003d requests.codes.ok:"},{"line_number":84,"context_line":"        if len(response.content) \u003d\u003d 0:"},{"line_number":85,"context_line":"            raise exception.IronicException(_("},{"line_number":86,"context_line":"                \"Configuration mold for node %s at %s is empty\")"},{"line_number":87,"context_line":"                % (task.node.uuid, url))"},{"line_number":88,"context_line":"        try:"},{"line_number":89,"context_line":"            return response.json()"}],"source_content_type":"text/x-python","patch_set":1,"id":"d496235d_465e0c2d","line":86,"updated":"2021-08-20 11:57:54.000000000","message":"using more than one %s is not translation-friendly (since they may be reordered in the translated text). please use %(node)s, %(url)s","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"1e11b0a39971659ac091657791465a633d2a4135","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    if response.status_code \u003d\u003d requests.codes.ok:"},{"line_number":84,"context_line":"        if len(response.content) \u003d\u003d 0:"},{"line_number":85,"context_line":"            raise exception.IronicException(_("},{"line_number":86,"context_line":"                \"Configuration mold for node %s at %s is empty\")"},{"line_number":87,"context_line":"                % (task.node.uuid, url))"},{"line_number":88,"context_line":"        try:"},{"line_number":89,"context_line":"            return response.json()"}],"source_content_type":"text/x-python","patch_set":1,"id":"0c0c21e2_c4b044f5","line":86,"in_reply_to":"d496235d_465e0c2d","updated":"2021-08-23 10:14:05.000000000","message":"done","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"64575ef271aff6729c66cefb0a0f96b2ad528a32","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            return response.json()"},{"line_number":90,"context_line":"        except json.decoder.JSONDecodeError as jde:"},{"line_number":91,"context_line":"            raise exception.IronicException(_("},{"line_number":92,"context_line":"                \"Configuration mold for node %s at %s has invalid JSON: %s\")"},{"line_number":93,"context_line":"                % (task.node.uuid, url, jde))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    response.raise_for_status()"}],"source_content_type":"text/x-python","patch_set":1,"id":"16010dbb_0767574a","line":92,"updated":"2021-08-20 11:57:54.000000000","message":"same","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"1e11b0a39971659ac091657791465a633d2a4135","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            return response.json()"},{"line_number":90,"context_line":"        except json.decoder.JSONDecodeError as jde:"},{"line_number":91,"context_line":"            raise exception.IronicException(_("},{"line_number":92,"context_line":"                \"Configuration mold for node %s at %s has invalid JSON: %s\")"},{"line_number":93,"context_line":"                % (task.node.uuid, url, jde))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    response.raise_for_status()"}],"source_content_type":"text/x-python","patch_set":1,"id":"6919898f_fa59e64b","line":92,"in_reply_to":"16010dbb_0767574a","updated":"2021-08-23 10:14:05.000000000","message":"done","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"}],"ironic/drivers/modules/drac/management.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"64575ef271aff6729c66cefb0a0f96b2ad528a32","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    try:"},{"line_number":215,"context_line":"        jsonschema.validate(data, _CONF_MOLD_SCHEMA)"},{"line_number":216,"context_line":"    except json_schema_exc.ValidationError as e:"},{"line_number":217,"context_line":"        msg \u003d _(\"Configuration mold validation error: %s\") % e.message"},{"line_number":218,"context_line":"        raise exception.InvalidParameterValue(msg)"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"add43f55_7a9b1947","line":217,"updated":"2021-08-20 11:57:54.000000000","message":"nit: I think you can/should use just \"e\", without \".message\"","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"1e11b0a39971659ac091657791465a633d2a4135","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    try:"},{"line_number":215,"context_line":"        jsonschema.validate(data, _CONF_MOLD_SCHEMA)"},{"line_number":216,"context_line":"    except json_schema_exc.ValidationError as e:"},{"line_number":217,"context_line":"        msg \u003d _(\"Configuration mold validation error: %s\") % e.message"},{"line_number":218,"context_line":"        raise exception.InvalidParameterValue(msg)"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5b6b4576_7df949ae","line":217,"in_reply_to":"add43f55_7a9b1947","updated":"2021-08-23 10:14:05.000000000","message":"done","commit_id":"b4bd011b47692447d99e2f4f58c6655734f85771"}]}