)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"0bf0e4034532e68d0f8aa14641e3d027fa428496","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"288d7951_22fad549","updated":"2026-01-14 12:17:58.000000000","message":"recheck","commit_id":"ad7ca76ebadbf0b09dd4ad445fd3e45ef57448f6"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"6a4e300255dc52ed8c2edde76bb44cc84ce9f4f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c654d721_f5decafb","updated":"2026-01-15 07:29:45.000000000","message":"recheck","commit_id":"ad7ca76ebadbf0b09dd4ad445fd3e45ef57448f6"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9883a17086516cf15ff55bc4400c3825d5ef772d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4296ddf5_71aa1315","updated":"2026-01-20 11:56:02.000000000","message":"the implementation lgtm, just left a couple of minor improvements","commit_id":"5602b3cc16aebfa1949f95c65e84f2859a723812"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"dd81248107fff9884e7d007697f47216775518bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3a873c27_76012997","updated":"2026-01-22 10:34:44.000000000","message":"lgtm, thanks for the changes!","commit_id":"f6aac33e7018383066d8bcaa56e9d127960696a0"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"fa03ae430a76601905839675b841d1a0afba1848","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"43864ece_7957c3c0","updated":"2026-01-22 16:18:36.000000000","message":"recheck","commit_id":"f6aac33e7018383066d8bcaa56e9d127960696a0"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"1a0a2134ffa091967eb33db895f9f99fdb5f4f08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5225fa67_ac1d3a98","updated":"2026-01-22 12:24:48.000000000","message":"recheck","commit_id":"f6aac33e7018383066d8bcaa56e9d127960696a0"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"052c3c8526ec204ae9fe04c553738ad761b5cf69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c0c1d7eb_0304c5f5","updated":"2026-01-21 19:50:13.000000000","message":"recheck\n\nlgtm, as we discussed before, we don\u0027t want these thread kill behaviors in the code. CI failure is not related to this patch to","commit_id":"f6aac33e7018383066d8bcaa56e9d127960696a0"}],"releasenotes/notes/blueprint-remove-cancel-ongoing-actions-11144f2acb562188.yaml":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9883a17086516cf15ff55bc4400c3825d5ef772d","unresolved":true,"context_lines":[{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Now, when cancelling a running action plan, only the **PENDING** actions"},{"line_number":5,"context_line":"    will be cancelled. The **ONGOING** actions will continue to run until they"},{"line_number":6,"context_line":"    are completed in all the cases."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"    For more details, see: https://blueprints.launchpad.net/watcher/+spec/remove-cancel-ongoing-actions."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"05075727_7e077165","line":5,"updated":"2026-01-20 11:56:02.000000000","message":"I\u0027m not sure if I understand correctly the second sentence, particularly the `in all the cases`. Is it trying to say that all ongoing actions will continue to run until they are finished? If so I think the last part could be removed and it would be a bit more clear \n```suggestion\n    will be cancelled. The **ONGOING** actions will continue to run until they finish.\n```","commit_id":"5602b3cc16aebfa1949f95c65e84f2859a723812"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"07337eadc0776190c7e6edbef32e8be871ab5043","unresolved":false,"context_lines":[{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Now, when cancelling a running action plan, only the **PENDING** actions"},{"line_number":5,"context_line":"    will be cancelled. The **ONGOING** actions will continue to run until they"},{"line_number":6,"context_line":"    are completed in all the cases."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"    For more details, see: https://blueprints.launchpad.net/watcher/+spec/remove-cancel-ongoing-actions."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0a1a694c_fc04e6c0","line":5,"in_reply_to":"05075727_7e077165","updated":"2026-01-21 17:20:43.000000000","message":"I agree, it\u0027s unclear. Fixing it.","commit_id":"5602b3cc16aebfa1949f95c65e84f2859a723812"}],"watcher/tests/unit/applier/workflow_engine/test_taskflow_action_container.py":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9883a17086516cf15ff55bc4400c3825d5ef772d","unresolved":true,"context_lines":[{"line_number":185,"context_line":""},{"line_number":186,"context_line":"    # Actions are cancelled when the action plan is in a cancelling"},{"line_number":187,"context_line":"    # state when entering the pre_execute method."},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"    def test_pre_execute_with_cancel_action_plan(self):"},{"line_number":190,"context_line":"        action_plan \u003d obj_utils.create_test_action_plan("},{"line_number":191,"context_line":"            self.context, audit_id\u003dself.audit.id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"8a5f1b00_7a2cc3dd","line":188,"updated":"2026-01-20 11:56:02.000000000","message":"I would remove this line to make it clear that the comment refers to the test case below, although I think it would be even better to move the coment inside the function body, maybe as a docstring","commit_id":"5602b3cc16aebfa1949f95c65e84f2859a723812"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"07337eadc0776190c7e6edbef32e8be871ab5043","unresolved":false,"context_lines":[{"line_number":185,"context_line":""},{"line_number":186,"context_line":"    # Actions are cancelled when the action plan is in a cancelling"},{"line_number":187,"context_line":"    # state when entering the pre_execute method."},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"    def test_pre_execute_with_cancel_action_plan(self):"},{"line_number":190,"context_line":"        action_plan \u003d obj_utils.create_test_action_plan("},{"line_number":191,"context_line":"            self.context, audit_id\u003dself.audit.id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"50383366_f7b8d0d9","line":188,"in_reply_to":"8a5f1b00_7a2cc3dd","updated":"2026-01-21 17:20:43.000000000","message":"Done","commit_id":"5602b3cc16aebfa1949f95c65e84f2859a723812"}]}