)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"5b5f0d892d72bb69806d9353ac57d57dee48538c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"aea7885c_333df6f7","updated":"2023-01-20 06:00:25.000000000","message":"recheck","commit_id":"fab56bbeb5bdba8a7e1d29619109abedd5af8a58"},{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"c1b2a414b05e5075dbaf431a75d6e757392fd8a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0aa3eb78_e1d6ab47","updated":"2023-01-23 05:43:49.000000000","message":"recheck","commit_id":"4fb95504530f550b870abde3261ac3f024fe8ca6"},{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"283359516ea92b39c53835de0c86fb70dc80f560","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ebcce13f_9bdd01fb","updated":"2023-01-20 12:53:38.000000000","message":"recheck","commit_id":"4fb95504530f550b870abde3261ac3f024fe8ca6"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"cb96b86b6193f7ccd42c273692638dc5cfe8b334","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"7f3adfd0_4b624faa","updated":"2023-01-24 21:11:29.000000000","message":"Please add unit tests, docs about how to enable it and edit your commit message because it looks like this feature is enabled by default","commit_id":"9ed628e8ddd1a304beaef4f17d2e1b0a2f016989"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"7fb0823f111c5fe931d4202e5c2543f9f1adf754","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7e776137_c586864d","updated":"2023-02-03 08:19:11.000000000","message":"also I\u0027d like to ask about unit tests, but actially I have no idea how to override non-mistral parameters in tests, if someone knows, please help, otherwise I\u0027m ok merging it without dedicated unit tests.","commit_id":"e290bc2839ae0fbb5f66f9bc0397ba6102077d27"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"ea2aa31cea37e1ba555a544683f7dbddef3f7ff9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"681037c3_20f65c25","updated":"2023-02-02 21:00:57.000000000","message":"take a look at the resulting doc:\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c66/871025/10/check/openstack-tox-docs/c66e0e8/docs/admin/configuration/config-guide.html\n\nthere is an extra _ (underscore) before official Oslo log","commit_id":"e290bc2839ae0fbb5f66f9bc0397ba6102077d27"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"3280ea54d86e56e60d64e336d65fa6b6d6e609d4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"09797c4d_8d6ada96","updated":"2023-02-02 20:57:57.000000000","message":"why is it named trace_uuid while it present the root execution_id?\nwhy not keeping root_execution_id?","commit_id":"e290bc2839ae0fbb5f66f9bc0397ba6102077d27"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"bcc9e4b27a208e0f5524e3b0bb671e12f467314e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"bcc12afc_f7360161","in_reply_to":"09797c4d_8d6ada96","updated":"2023-02-03 08:17:49.000000000","message":"good one, it is up to client how to configure it for logging, but root_execution_id is more suitable for variable in context","commit_id":"e290bc2839ae0fbb5f66f9bc0397ba6102077d27"},{"author":{"_account_id":35600,"name":"Axel Vanzaghi","display_name":"Axel Vanzaghi","email":"avanzaghi.osf@axellink.fr","username":"axel.vanzaghi","status":"OVHcloud"},"change_message_id":"80af603067ce69c02e2144d7d34cadd1e4c5a02f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3090d3b5_05cac612","updated":"2023-02-07 08:41:07.000000000","message":"I will agree With Arnaud Morin in that there is no need to rename root_execution_id to trace_uuid, please resolve this.\nOther that that, change is good.","commit_id":"c93801b6bf3afc75e4abfe72cda107b001070b02"},{"author":{"_account_id":35600,"name":"Axel Vanzaghi","display_name":"Axel Vanzaghi","email":"avanzaghi.osf@axellink.fr","username":"axel.vanzaghi","status":"OVHcloud"},"change_message_id":"1690717de347fcf7eee5f56c963f24617ba37ab3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"a24599d9_06bffd58","updated":"2023-02-09 09:27:07.000000000","message":"LGTM","commit_id":"9efbdf040fab7b3e50c6f54e5f8981b4364891eb"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"122bcc855fae07694c87e89388cc96ff8b8f0fb1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"fb952bdc_a06d274b","updated":"2023-02-09 18:41:31.000000000","message":"LGTM as well, but I think you forgot to rename it in the commit message itself!\n","commit_id":"9efbdf040fab7b3e50c6f54e5f8981b4364891eb"},{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"fad10193bb849e0ecbb7e90bda074f420101b812","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"931babdd_88333289","updated":"2023-02-14 08:01:42.000000000","message":"recheck","commit_id":"88e7e7ceee6079afd52dffaad7ace006803606a3"},{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"43128c634473b57042502e9b63b537896aebc777","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"cb7166f6_fd8bb7a1","updated":"2023-02-13 07:29:45.000000000","message":"recheck","commit_id":"88e7e7ceee6079afd52dffaad7ace006803606a3"}],"mistral/utils/wf_trace.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"3280ea54d86e56e60d64e336d65fa6b6d6e609d4","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        exec_id \u003d obj.workflow_execution_id"},{"line_number":38,"context_line":"        task_id \u003d obj.id"},{"line_number":39,"context_line":"        debug_info \u003d \u0027(wf_ex_id\u003d%s, task_ex_id\u003d%s)\u0027 % (exec_id, task_id)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    elif type(obj) is models.WorkflowExecution:"},{"line_number":42,"context_line":"        debug_info \u003d \u0027(execution_id\u003d%s,\u0027 \\"},{"line_number":43,"context_line":"                     \u0027task_execution_id\u003d%s)\u0027 % (obj.id, obj.task_execution_id)"}],"source_content_type":"text/x-python","patch_set":10,"id":"81f279c1_61aa86d7","line":40,"updated":"2023-02-02 20:57:57.000000000","message":"Should it be consistent? task_ex_id here and task_execution_id few lines below","commit_id":"e290bc2839ae0fbb5f66f9bc0397ba6102077d27"}]}