)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"65a43ff9ac80facdb2859af627ca7a5f343baf28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"66c2655c_f8d3a9aa","updated":"2024-07-24 20:00:45.000000000","message":"Tests","commit_id":"d1a79e0ed274fa611222b35d763754758f2b63a8"}],"distributedcloud/dcmanager/api/controllers/v1/sw_update_strategy.py":[{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"ac93861b8b6310079ea2ad90e1afc74d582bbfef","unresolved":true,"context_lines":[{"line_number":166,"context_line":"                ]:"},{"line_number":167,"context_line":"                    pecan.abort(400, _(\"subcloud-apply-type invalid\"))"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"            patch_id \u003d payload.get(\"patch_id\")"},{"line_number":170,"context_line":"            if strategy_type \u003d\u003d consts.SW_UPDATE_TYPE_PATCH and not patch_id:"},{"line_number":171,"context_line":"                message \u003d ("},{"line_number":172,"context_line":"                    f\"patch_id parameter is required for {strategy_type} strategy.\""},{"line_number":173,"context_line":"                )"},{"line_number":174,"context_line":"                pecan.abort(400, _(message))"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"            if strategy_type \u003d\u003d consts.SW_UPDATE_TYPE_SOFTWARE and not payload.get("},{"line_number":177,"context_line":"                \"release_id\""},{"line_number":178,"context_line":"            ):"}],"source_content_type":"text/x-python","patch_set":5,"id":"96a6852d_6bfdbf8b","line":175,"range":{"start_line":169,"start_character":0,"end_line":175,"end_character":0},"updated":"2024-07-30 18:41:03.000000000","message":"Should also check for the existence of the patch file given the ID here","commit_id":"0be2a34e43d2e1b941c58d92fb0d8c15a2439e9e"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"02ad01a63de6cda94f66ad58a6ef2af604b8255e","unresolved":false,"context_lines":[{"line_number":166,"context_line":"                ]:"},{"line_number":167,"context_line":"                    pecan.abort(400, _(\"subcloud-apply-type invalid\"))"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"            patch_id \u003d payload.get(\"patch_id\")"},{"line_number":170,"context_line":"            if strategy_type \u003d\u003d consts.SW_UPDATE_TYPE_PATCH and not patch_id:"},{"line_number":171,"context_line":"                message \u003d ("},{"line_number":172,"context_line":"                    f\"patch_id parameter is required for {strategy_type} strategy.\""},{"line_number":173,"context_line":"                )"},{"line_number":174,"context_line":"                pecan.abort(400, _(message))"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"            if strategy_type \u003d\u003d consts.SW_UPDATE_TYPE_SOFTWARE and not payload.get("},{"line_number":177,"context_line":"                \"release_id\""},{"line_number":178,"context_line":"            ):"}],"source_content_type":"text/x-python","patch_set":5,"id":"c2cef4d3_8be8aefd","line":175,"range":{"start_line":169,"start_character":0,"end_line":175,"end_character":0},"in_reply_to":"96a6852d_6bfdbf8b","updated":"2024-07-30 19:41:15.000000000","message":"Done","commit_id":"0be2a34e43d2e1b941c58d92fb0d8c15a2439e9e"}],"distributedcloud/dcmanager/orchestrator/states/patch/updating_patches.py":[{"author":{"_account_id":36476,"name":"Raphael Lima","display_name":"Raphael","email":"Raphael.Lima@windriver.com","username":"rlima"},"change_message_id":"fc52d0f2a263fb93398e1433b097b4d8cef474c1","unresolved":true,"context_lines":[{"line_number":34,"context_line":"    def upload_patch(self, patch_id, strategy_step):"},{"line_number":35,"context_line":"        \"\"\"Upload a patch file to the subcloud\"\"\""},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        patch_file \u003d \"%s/%s/%s.patch\" % ("},{"line_number":38,"context_line":"            consts.PATCH_VAULT_DIR,"},{"line_number":39,"context_line":"            consts.PATCHING_SW_VERSION,"},{"line_number":40,"context_line":"            patch_id,"},{"line_number":41,"context_line":"        )"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        if not os.path.isfile(patch_file):"},{"line_number":44,"context_line":"            message \u003d f\"Patch file {patch_file} is missing\""}],"source_content_type":"text/x-python","patch_set":4,"id":"204509b1_1466c395","line":41,"range":{"start_line":37,"start_character":21,"end_line":41,"end_character":9},"updated":"2024-07-30 18:05:12.000000000","message":"Use f-string","commit_id":"c3a1989ab30e9bea0a67ab2d3b7b39dcfa1712a1"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"c7c348183f71c38d2cf3a80e576b90d91425084b","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    def upload_patch(self, patch_id, strategy_step):"},{"line_number":35,"context_line":"        \"\"\"Upload a patch file to the subcloud\"\"\""},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        patch_file \u003d \"%s/%s/%s.patch\" % ("},{"line_number":38,"context_line":"            consts.PATCH_VAULT_DIR,"},{"line_number":39,"context_line":"            consts.PATCHING_SW_VERSION,"},{"line_number":40,"context_line":"            patch_id,"},{"line_number":41,"context_line":"        )"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        if not os.path.isfile(patch_file):"},{"line_number":44,"context_line":"            message \u003d f\"Patch file {patch_file} is missing\""}],"source_content_type":"text/x-python","patch_set":4,"id":"f14b3e0d_71c0dc4c","line":41,"range":{"start_line":37,"start_character":21,"end_line":41,"end_character":9},"in_reply_to":"204509b1_1466c395","updated":"2024-07-30 18:14:56.000000000","message":"Done","commit_id":"c3a1989ab30e9bea0a67ab2d3b7b39dcfa1712a1"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"ac93861b8b6310079ea2ad90e1afc74d582bbfef","unresolved":true,"context_lines":[{"line_number":39,"context_line":"            f\"{patch_id}.patch\""},{"line_number":40,"context_line":"        )"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        if not os.path.isfile(patch_file):"},{"line_number":43,"context_line":"            message \u003d f\"Patch file {patch_file} is missing\""},{"line_number":44,"context_line":"            self.error_log(strategy_step, message)"},{"line_number":45,"context_line":"            raise Exception(message)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        self.info_log("},{"line_number":48,"context_line":"            strategy_step,"},{"line_number":49,"context_line":"            f\"Patch {patch_id} will be uploaded to subcloud\","}],"source_content_type":"text/x-python","patch_set":5,"id":"4f7c073d_90254144","line":46,"range":{"start_line":42,"start_character":0,"end_line":46,"end_character":0},"updated":"2024-07-30 18:41:03.000000000","message":"The API should have this check","commit_id":"0be2a34e43d2e1b941c58d92fb0d8c15a2439e9e"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"02ad01a63de6cda94f66ad58a6ef2af604b8255e","unresolved":false,"context_lines":[{"line_number":39,"context_line":"            f\"{patch_id}.patch\""},{"line_number":40,"context_line":"        )"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        if not os.path.isfile(patch_file):"},{"line_number":43,"context_line":"            message \u003d f\"Patch file {patch_file} is missing\""},{"line_number":44,"context_line":"            self.error_log(strategy_step, message)"},{"line_number":45,"context_line":"            raise Exception(message)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        self.info_log("},{"line_number":48,"context_line":"            strategy_step,"},{"line_number":49,"context_line":"            f\"Patch {patch_id} will be uploaded to subcloud\","}],"source_content_type":"text/x-python","patch_set":5,"id":"3778b4bd_ce6a6575","line":46,"range":{"start_line":42,"start_character":0,"end_line":46,"end_character":0},"in_reply_to":"4f7c073d_90254144","updated":"2024-07-30 19:41:15.000000000","message":"Done","commit_id":"0be2a34e43d2e1b941c58d92fb0d8c15a2439e9e"}]}