)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b0251584e0d919774332c1aed89194fafc36f819","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ca4e651a_6dc282c2","updated":"2026-03-03 08:41:10.000000000","message":"@smooney@redhat.com do we need a release note for requirement.txt bump?","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3c24e87b3c35d4b37154bb6d16738c78ddb38169","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b41f021b_4cb1704f","updated":"2026-03-03 10:49:03.000000000","message":"this is workable in its current form.\n\nthere are some changes to how this code is stucture that iw ould like to make but as part of the seperate seies i started so ill ignore those for now","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3c24e87b3c35d4b37154bb6d16738c78ddb38169","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5a708e93_3eb0f846","in_reply_to":"ca4e651a_6dc282c2","updated":"2026-03-03 10:49:03.000000000","message":"no as long as its allowed by upper constraits already we do not require release notes for increaing deps. we could have them for addign or removing deps but its more nice to have rather than required even in that case.\n\nrelease notes are primarly for the operator not packagers so we rarely document dependency in release notes","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a34bb3598180f59eec5e659b0c41f8a94d4a9f2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c8a9842c_e5814cdf","updated":"2026-03-03 19:29:49.000000000","message":"i agree with some fo the comments Doug left inline but i think we should proceed with this and address them next cycle","commit_id":"0561c24b445af8be5bdada78a566fe80ed48fc37"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e825c2f46312886c1edb9947e7e14151af7d27d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7dd3510f_d1eb79c7","updated":"2026-03-03 19:22:34.000000000","message":"lgtm, thanks","commit_id":"0561c24b445af8be5bdada78a566fe80ed48fc37"}],"watcher_dashboard/api/watcher.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e30afd3e8d40368a060d7335f1f87b0682677607","unresolved":false,"context_lines":[{"line_number":451,"context_line":"            patch.append("},{"line_number":452,"context_line":"                {\u0027op\u0027: \u0027replace\u0027, \u0027path\u0027: \u0027/state\u0027,"},{"line_number":453,"context_line":"                 \u0027value\u0027: state})"},{"line_number":454,"context_line":"        if reason is not None and reason !\u003d \"\":"},{"line_number":455,"context_line":"            patch.append("},{"line_number":456,"context_line":"                {\u0027op\u0027: \u0027replace\u0027, \u0027path\u0027: \u0027/status_message\u0027,"},{"line_number":457,"context_line":"                 \u0027value\u0027: reason})"}],"source_content_type":"text/x-python","patch_set":2,"id":"82790a8f_e4fd919e","line":454,"in_reply_to":"818add45_64881444","updated":"2026-03-03 11:04:34.000000000","message":"Done","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3c24e87b3c35d4b37154bb6d16738c78ddb38169","unresolved":true,"context_lines":[{"line_number":451,"context_line":"            patch.append("},{"line_number":452,"context_line":"                {\u0027op\u0027: \u0027replace\u0027, \u0027path\u0027: \u0027/state\u0027,"},{"line_number":453,"context_line":"                 \u0027value\u0027: state})"},{"line_number":454,"context_line":"        if reason is not None and reason !\u003d \"\":"},{"line_number":455,"context_line":"            patch.append("},{"line_number":456,"context_line":"                {\u0027op\u0027: \u0027replace\u0027, \u0027path\u0027: \u0027/status_message\u0027,"},{"line_number":457,"context_line":"                 \u0027value\u0027: reason})"}],"source_content_type":"text/x-python","patch_set":2,"id":"818add45_64881444","line":454,"in_reply_to":"d54753f3_865ba32c","updated":"2026-03-03 10:49:03.000000000","message":"this is a reasonable observartion.\nwe could add `reason \u003d reason or None`\n\nat the start of the function and then just make this \n`if reason is not None:`","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e825c2f46312886c1edb9947e7e14151af7d27d7","unresolved":true,"context_lines":[{"line_number":389,"context_line":"        self._request \u003d request"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"    @classmethod"},{"line_number":392,"context_line":"    def list(cls, request, **filters):"},{"line_number":393,"context_line":"        \"\"\"Return a list of actions in Watcher"},{"line_number":394,"context_line":""},{"line_number":395,"context_line":"        :param request: request object"}],"source_content_type":"text/x-python","patch_set":3,"id":"a81fd27a_0b2b7921","line":392,"range":{"start_line":392,"start_character":0,"end_line":392,"end_character":38},"updated":"2026-03-03 19:22:34.000000000","message":"consider adding api_version\u003dNone to these methods too, in a follow up.","commit_id":"0561c24b445af8be5bdada78a566fe80ed48fc37"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e825c2f46312886c1edb9947e7e14151af7d27d7","unresolved":true,"context_lines":[{"line_number":469,"context_line":"            return None"},{"line_number":470,"context_line":""},{"line_number":471,"context_line":"    @classmethod"},{"line_number":472,"context_line":"    def delete(cls, request, action_id):"},{"line_number":473,"context_line":"        \"\"\"Delete an action"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        :param request: request object"}],"source_content_type":"text/x-python","patch_set":3,"id":"8ae4dafb_907119f5","line":472,"range":{"start_line":472,"start_character":8,"end_line":472,"end_character":39},"updated":"2026-03-03 19:22:34.000000000","message":"consider adding api_version\u003dNone to these methods too, in a follow up.","commit_id":"0561c24b445af8be5bdada78a566fe80ed48fc37"}],"watcher_dashboard/common/exceptions.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3c24e87b3c35d4b37154bb6d16738c78ddb38169","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class WatcherDashboardException(Exception):"},{"line_number":27,"context_line":"    \"\"\"Base exception for watcher-dashboard errors.\"\"\""},{"line_number":28,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"b841471a_b4c7bbb9","line":28,"range":{"start_line":28,"start_character":2,"end_line":28,"end_character":8},"updated":"2026-03-03 10:49:03.000000000","message":"nit technially pass is not required the doc string is enough\nto declare a empty child class.\n\n\n```\n\u003e\u003e\u003e class MyExcpetion(Exception):\n...     \"\"\" test \"\"\"\n...\n\u003e\u003e\u003e dir(MyExcpetion)\n[\u0027__cause__\u0027, \u0027__class__\u0027, \u0027__context__\u0027, \u0027__delattr__\u0027, \u0027__dict__\u0027, \u0027__dir__\u0027, \u0027__doc__\u0027, \u0027__eq__\u0027, \u0027__firstlineno__\u0027, \u0027__format__\u0027, \u0027__ge__\u0027, \u0027__getattribute__\u0027, \u0027__getstate__\u0027, \u0027__gt__\u0027, \u0027__hash__\u0027, \u0027__init__\u0027, \u0027__init_subclass__\u0027, \u0027__le__\u0027, \u0027__lt__\u0027, \u0027__module__\u0027, \u0027__ne__\u0027, \u0027__new__\u0027, \u0027__reduce__\u0027, \u0027__reduce_ex__\u0027, \u0027__repr__\u0027, \u0027__setattr__\u0027, \u0027__setstate__\u0027, \u0027__sizeof__\u0027, \u0027__static_attributes__\u0027, \u0027__str__\u0027, \u0027__subclasshook__\u0027, \u0027__suppress_context__\u0027, \u0027__traceback__\u0027, \u0027__weakref__\u0027, \u0027add_note\u0027, \u0027args\u0027, \u0027with_traceback\u0027]\n\u003e\u003e\u003e str(MyExcpetion)\n\"\u003cclass \u0027__main__.MyExcpetion\u0027\u003e\"\n\u003e\u003e\u003e\n```","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a34bb3598180f59eec5e659b0c41f8a94d4a9f2c","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class WatcherDashboardException(Exception):"},{"line_number":27,"context_line":"    \"\"\"Base exception for watcher-dashboard errors.\"\"\""},{"line_number":28,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"76de03eb_55e27258","line":28,"range":{"start_line":28,"start_character":2,"end_line":28,"end_character":8},"in_reply_to":"1b65474b_c3d8368d","updated":"2026-03-03 19:29:49.000000000","message":"Acknowledged","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e30afd3e8d40368a060d7335f1f87b0682677607","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class WatcherDashboardException(Exception):"},{"line_number":27,"context_line":"    \"\"\"Base exception for watcher-dashboard errors.\"\"\""},{"line_number":28,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"1b65474b_c3d8368d","line":28,"range":{"start_line":28,"start_character":2,"end_line":28,"end_character":8},"in_reply_to":"b841471a_b4c7bbb9","updated":"2026-03-03 11:04:34.000000000","message":"Ah, I was not aware about that. Thank you!","commit_id":"1fc78e07655f1f12bafd282f647e060415036b26"}],"watcher_dashboard/templates/infra_optim/actions/details.html":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e825c2f46312886c1edb9947e7e14151af7d27d7","unresolved":true,"context_lines":[{"line_number":23,"context_line":"        \u003cdt\u003e{% trans \"State\" %}\u003c/dt\u003e"},{"line_number":24,"context_line":"        \u003cdd\u003e{{ action.state|default:_(\"-\") }}\u003c/dd\u003e"},{"line_number":25,"context_line":"        {% if action.status_message %}"},{"line_number":26,"context_line":"        \u003cdt\u003e{% if action.state \u003d\u003d \"SKIPPED\" %}{% trans \"Skip Reason\" %}{% else %}{% trans \"Status Message\" %}{% endif %}\u003c/dt\u003e"},{"line_number":27,"context_line":"        \u003cdd\u003e{{ action.status_message }}\u003c/dd\u003e"},{"line_number":28,"context_line":"        {% endif %}"},{"line_number":29,"context_line":"        \u003cdt\u003e{% trans \"Created At\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":3,"id":"525f51ab_f7e43a57","line":26,"range":{"start_line":26,"start_character":47,"end_line":26,"end_character":109},"updated":"2026-03-03 19:22:34.000000000","message":"nit: I don\u0027t think that is really needed, I would simplify to \"More Details\" or \"Status Message\", since the message itself already says that was skipped (\"Action was skipped automatically: %s\" or \"Action skipped by user\")","commit_id":"0561c24b445af8be5bdada78a566fe80ed48fc37"}]}