)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8fb900dafb66f3605f8bee13ca9a8e06e78f1a82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"358121c8_e655db1d","updated":"2026-02-26 15:04:27.000000000","message":"i think this is good.\nif you decied to add the lenght check and truncation please also add a test.","commit_id":"ccad87ec7c86b525544d663126857ea2bd8abd74"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"7c059c74ccf0d3ff396de240ea335eb43e98dd83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9157636c_6c0a71d5","updated":"2026-02-26 15:31:11.000000000","message":"lgtm, thanks","commit_id":"ccad87ec7c86b525544d663126857ea2bd8abd74"}],"watcher/applier/workflow_engine/base.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8fb900dafb66f3605f8bee13ca9a8e06e78f1a82","unresolved":true,"context_lines":[{"line_number":153,"context_line":"        kwargs \u003d {}"},{"line_number":154,"context_line":"        if reason:"},{"line_number":155,"context_line":"            kwargs[\"status_message\"] \u003d (_("},{"line_number":156,"context_line":"                \"Action failed in %s: %s\") % (phase, reason))"},{"line_number":157,"context_line":"        db_action \u003d self.engine.notify(self._db_action,"},{"line_number":158,"context_line":"                                       objects.action.State.FAILED,"},{"line_number":159,"context_line":"                                       **kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"378dc01d_fc2d5291","line":156,"updated":"2026-02-26 15:04:27.000000000","message":"if we want to make this bullet prrof you coudl do a lenght check or truncation here.\n\nbut i think this is ok as is","commit_id":"ccad87ec7c86b525544d663126857ea2bd8abd74"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8fb900dafb66f3605f8bee13ca9a8e06e78f1a82","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        notifications.action.send_execution_notification("},{"line_number":161,"context_line":"            self.engine.context, db_action,"},{"line_number":162,"context_line":"            fields.NotificationAction.EXECUTION,"},{"line_number":163,"context_line":"            fields.NotificationPhase.ERROR,"},{"line_number":164,"context_line":"            priority\u003dfields.NotificationPriority.ERROR)"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    # NOTE(alexchadin): taskflow does 3 method calls (pre_execute, execute,"}],"source_content_type":"text/x-python","patch_set":1,"id":"8b03fc0d_c39ef370","line":163,"in_reply_to":"734b4bc9_f49b433d","updated":"2026-02-26 15:04:27.000000000","message":"no this is fine as is","commit_id":"ccad87ec7c86b525544d663126857ea2bd8abd74"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"7c059c74ccf0d3ff396de240ea335eb43e98dd83","unresolved":false,"context_lines":[{"line_number":199,"context_line":"            notifications.action.send_update("},{"line_number":200,"context_line":"                self.engine.context, db_action,"},{"line_number":201,"context_line":"                old_state\u003dobjects.action.State.PENDING)"},{"line_number":202,"context_line":"        except exception.WatcherException as e:"},{"line_number":203,"context_line":"            LOG.exception(e)"},{"line_number":204,"context_line":"            self._fail_action(\"pre_condition\", reason\u003dstr(e))"},{"line_number":205,"context_line":"        except Exception as e:"},{"line_number":206,"context_line":"            LOG.exception(e)"},{"line_number":207,"context_line":"            self._fail_action(\"pre_condition\", reason\u003dtype(e).__name__)"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    def execute(self, *args, **kwargs):"},{"line_number":210,"context_line":"        action_object \u003d objects.Action.get_by_uuid("}],"source_content_type":"text/x-python","patch_set":1,"id":"7cb9e09b_7ce33cae","line":207,"range":{"start_line":202,"start_character":0,"end_line":207,"end_character":71},"updated":"2026-02-26 15:31:11.000000000","message":"+1","commit_id":"ccad87ec7c86b525544d663126857ea2bd8abd74"}],"watcher/tests/unit/applier/workflow_engine/test_taskflow_action_container.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8fb900dafb66f3605f8bee13ca9a8e06e78f1a82","unresolved":true,"context_lines":[{"line_number":149,"context_line":"    def test_pre_execute_with_failed_exception(self, m_pre_condition):"},{"line_number":150,"context_line":"        # When failed with a non-watcher exception, the status_message"},{"line_number":151,"context_line":"        # should only include the type of the exception."},{"line_number":152,"context_line":"        m_pre_condition.side_effect \u003d Exception(\"Third party exception\")"},{"line_number":153,"context_line":"        action_plan \u003d obj_utils.create_test_action_plan("},{"line_number":154,"context_line":"            self.context, audit_id\u003dself.audit.id,"},{"line_number":155,"context_line":"            strategy_id\u003dself.strategy.id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"97b298fb_f32875e8","line":152,"range":{"start_line":152,"start_character":38,"end_line":152,"end_character":47},"updated":"2026-02-26 15:04:27.000000000","message":"nit value error or similar would make ti clear that this the type below in the assert.\n\nbut this is fine","commit_id":"ccad87ec7c86b525544d663126857ea2bd8abd74"}]}