)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"fa2775a37050f6d7c578bcb18c00ae1cad458b85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e47c1510_ce282632","updated":"2024-01-23 12:04:53.000000000","message":"Do you plan to continue to work on it?","commit_id":"5ec7a372c39513fb1217f60375d89d51eaa1a75f"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"ab2f6c066e0952adab9bf0906d857ac44c1eda1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ebe9070b_cb077976","in_reply_to":"e47c1510_ce282632","updated":"2024-01-23 14:56:44.000000000","message":"Hi, I may give it a try.\nRight now, I would say that the fix included in the patch is a dirty workaround (but maybe it\u0027s better to have a dirty workaround than a nasty bug). A \"real\" fix would be more complicated.\n\nFYI the patch is related to\nhttps://bugs.launchpad.net/taskflow/+bug/2043808\nhttps://bugs.launchpad.net/octavia/+bug/2043360","commit_id":"5ec7a372c39513fb1217f60375d89d51eaa1a75f"}],"taskflow/engines/action_engine/runtime.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"cccbe4def179c630dc22879f8c1b09b7d7775762","unresolved":true,"context_lines":[{"line_number":301,"context_line":"            # flow may still be running in another thread. If this task"},{"line_number":302,"context_line":"            # triggers a RETRY, it overrides the previously set REVERT status,"},{"line_number":303,"context_line":"            # breaking the revert path of the flow."},{"line_number":304,"context_line":"            if cur_intention \u003d\u003d st.REVERT and intention \u003d\u003d st.RETRY:"},{"line_number":305,"context_line":"                continue"},{"line_number":306,"context_line":"            if state or intention:"},{"line_number":307,"context_line":"                tweaked.append((atom, state, intention))"}],"source_content_type":"text/x-python","patch_set":2,"id":"52608cc6_db2b4574","line":304,"range":{"start_line":304,"start_character":12,"end_line":304,"end_character":68},"updated":"2024-08-07 12:20:45.000000000","message":"maybe this can be\n\n```\nif cur_intention in [st.REVERT, st.RETRY]:\n```","commit_id":"0cb423b64209f9412ebe20fb562b916fa09c5cbc"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"bc47187006570fcd1c9c9ef268680e77c2c88cae","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            # flow may still be running in another thread. If this task"},{"line_number":302,"context_line":"            # triggers a RETRY, it overrides the previously set REVERT status,"},{"line_number":303,"context_line":"            # breaking the revert path of the flow."},{"line_number":304,"context_line":"            if cur_intention \u003d\u003d st.REVERT and intention \u003d\u003d st.RETRY:"},{"line_number":305,"context_line":"                continue"},{"line_number":306,"context_line":"            if state or intention:"},{"line_number":307,"context_line":"                tweaked.append((atom, state, intention))"}],"source_content_type":"text/x-python","patch_set":2,"id":"d1c1b061_bb4f4040","line":304,"range":{"start_line":304,"start_character":12,"end_line":304,"end_character":68},"in_reply_to":"3769ddaf_ffa36117","updated":"2024-08-07 12:39:02.000000000","message":"Oops. You are correct !","commit_id":"0cb423b64209f9412ebe20fb562b916fa09c5cbc"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"3d7b693fe5508d30ced3462b3ea544dae16aecd4","unresolved":true,"context_lines":[{"line_number":301,"context_line":"            # flow may still be running in another thread. If this task"},{"line_number":302,"context_line":"            # triggers a RETRY, it overrides the previously set REVERT status,"},{"line_number":303,"context_line":"            # breaking the revert path of the flow."},{"line_number":304,"context_line":"            if cur_intention \u003d\u003d st.REVERT and intention \u003d\u003d st.RETRY:"},{"line_number":305,"context_line":"                continue"},{"line_number":306,"context_line":"            if state or intention:"},{"line_number":307,"context_line":"                tweaked.append((atom, state, intention))"}],"source_content_type":"text/x-python","patch_set":2,"id":"3769ddaf_ffa36117","line":304,"range":{"start_line":304,"start_character":12,"end_line":304,"end_character":68},"in_reply_to":"52608cc6_db2b4574","updated":"2024-08-07 12:31:22.000000000","message":"it\u0027s not the same, there\u0027s \"cur_intention\" and \"intention\" variables there","commit_id":"0cb423b64209f9412ebe20fb562b916fa09c5cbc"}]}