)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"73050aebea4fa058293d0abbe5376d0c5b3b95f4","unresolved":true,"context_lines":[{"line_number":12,"context_line":"ID - in our example, the request ID of the original POST /servers"},{"line_number":13,"context_line":"request."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch makes create_server() return the request ID, and wires that"},{"line_number":16,"context_line":"value up into wait_for_server_status(), so that the latter can log it"},{"line_number":17,"context_line":"if the wait fails."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9b7a7e14_ace45415","line":15,"range":{"start_line":15,"start_character":11,"end_line":15,"end_character":39},"updated":"2022-01-13 10:48:07.000000000","message":"return value of create_server is not changed, the commit just makes the code parse one more value out of the create_server\u0027s response","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"198508f633754411de2af81f8a396855b8f38ab8","unresolved":false,"context_lines":[{"line_number":12,"context_line":"ID - in our example, the request ID of the original POST /servers"},{"line_number":13,"context_line":"request."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch makes create_server() return the request ID, and wires that"},{"line_number":16,"context_line":"value up into wait_for_server_status(), so that the latter can log it"},{"line_number":17,"context_line":"if the wait fails."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"e8b8316e_0d8c0e21","line":15,"range":{"start_line":15,"start_character":11,"end_line":15,"end_character":39},"in_reply_to":"9b7a7e14_ace45415","updated":"2022-01-13 21:12:26.000000000","message":"Done","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"17fe3a2600b7b71efd15fcfc2b0c732ad283858e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5fa12128_271fbef5","updated":"2021-11-23 14:49:25.000000000","message":"Since of course the gate works perfectly the moment we want to test error code, I\u0027ve pushed a DNM nova patch here to explode instance builds and trigger this new req ID reporting code:\n\nhttps://review.opendev.org/c/openstack/nova/+/818932","commit_id":"49d700538e440b655c683802373c8bbc46e151bb"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"198508f633754411de2af81f8a396855b8f38ab8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fb4a4fd9_c287501a","in_reply_to":"5fa12128_271fbef5","updated":"2022-01-13 21:12:26.000000000","message":"\u003e Since of course the gate works perfectly the moment we want to test error code, I\u0027ve pushed a DNM nova patch here to explode instance builds and trigger this new req ID reporting code:\n\u003e \n\u003e https://review.opendev.org/c/openstack/nova/+/818932\n\nLooks like it\u0027s working as intended:\n\n  tempest.exceptions.BuildErrorException: Server b21d6cb7-a943-4aad-8eb8-75fd4c40bd3c failed to build and is in ERROR status\nDetails: Fault: {\u0027code\u0027: 500, \u0027created\u0027: \u00272021-11-23T17:40:36Z\u0027, \u0027message\u0027: \u0027boom\u0027}.Original request ID: req-2e3e8c28-6019-440a-a44c-afea6717dd92.\n\nAnd that\u0027s indeed the correct request ID in the nova-compute logs.","commit_id":"49d700538e440b655c683802373c8bbc46e151bb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"11949a716b2d2ba16b548f074e0a0ea7e014feba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"88f0b7b1_8922af31","updated":"2022-01-13 05:43:54.000000000","message":"idea lgtm, 1 comment on error message","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"73050aebea4fa058293d0abbe5376d0c5b3b95f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"41624d5d_b3c2371a","updated":"2022-01-13 10:48:07.000000000","message":"one small detail in the commit msg; i also agree this is a good idea","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"e2329db364a0fb03431068e8bdba12d65c5e591c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"67c6abbc_7c9e4e18","updated":"2022-01-16 16:00:03.000000000","message":"recheck","commit_id":"da48e4ee3d61088b6087adccbd0a70a8fd966231"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fec7bdd30971d39855465e57ad4c5e56db83bbb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6f0dba40_26a92282","updated":"2022-01-14 00:57:22.000000000","message":"recheck","commit_id":"da48e4ee3d61088b6087adccbd0a70a8fd966231"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d3efdeff8522ef4a08c257da562dbb15eb6eefef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"954ab890_2d2d4a59","updated":"2022-01-14 13:45:11.000000000","message":"recheck","commit_id":"da48e4ee3d61088b6087adccbd0a70a8fd966231"}],"tempest/common/waiters.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"11949a716b2d2ba16b548f074e0a0ea7e014feba","unresolved":true,"context_lines":[{"line_number":76,"context_line":"            if \u0027fault\u0027 in body:"},{"line_number":77,"context_line":"                details +\u003d \u0027Fault: %s.\u0027 % body[\u0027fault\u0027]"},{"line_number":78,"context_line":"            if request_id:"},{"line_number":79,"context_line":"                details +\u003d \u0027 Original request ID: %s.\u0027 % request_id"},{"line_number":80,"context_line":"            raise exceptions.BuildErrorException(details, server_id\u003dserver_id)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        timed_out \u003d int(time.time()) - start_time \u003e\u003d timeout"}],"source_content_type":"text/x-python","patch_set":3,"id":"adc283a9_addc1491","line":79,"range":{"start_line":79,"start_character":29,"end_line":79,"end_character":38},"updated":"2022-01-13 05:43:54.000000000","message":"\u0027Original request ID\u0027 is not so clear and confusing on what original means create or show or other. may be we can add something like\n\ndetails +\u003d \u0027 Server boot request ID: %s.\u0027 % request_id","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"198508f633754411de2af81f8a396855b8f38ab8","unresolved":false,"context_lines":[{"line_number":76,"context_line":"            if \u0027fault\u0027 in body:"},{"line_number":77,"context_line":"                details +\u003d \u0027Fault: %s.\u0027 % body[\u0027fault\u0027]"},{"line_number":78,"context_line":"            if request_id:"},{"line_number":79,"context_line":"                details +\u003d \u0027 Original request ID: %s.\u0027 % request_id"},{"line_number":80,"context_line":"            raise exceptions.BuildErrorException(details, server_id\u003dserver_id)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        timed_out \u003d int(time.time()) - start_time \u003e\u003d timeout"}],"source_content_type":"text/x-python","patch_set":3,"id":"8ac5e277_5f7eb3b8","line":79,"range":{"start_line":79,"start_character":29,"end_line":79,"end_character":38},"in_reply_to":"adc283a9_addc1491","updated":"2022-01-13 21:12:26.000000000","message":"Done","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"11949a716b2d2ba16b548f074e0a0ea7e014feba","unresolved":true,"context_lines":[{"line_number":91,"context_line":"                        \u0027expected_task_state\u0027: expected_task_state,"},{"line_number":92,"context_line":"                        \u0027timeout\u0027: timeout})"},{"line_number":93,"context_line":"            if request_id:"},{"line_number":94,"context_line":"                message +\u003d \u0027 Original request ID: %s.\u0027 % request_id"},{"line_number":95,"context_line":"            message +\u003d \u0027 Current status: %s.\u0027 % server_status"},{"line_number":96,"context_line":"            message +\u003d \u0027 Current task state: %s.\u0027 % task_state"},{"line_number":97,"context_line":"            caller \u003d test_utils.find_test_caller()"}],"source_content_type":"text/x-python","patch_set":3,"id":"873b6aad_607e0874","line":94,"range":{"start_line":94,"start_character":29,"end_line":94,"end_character":38},"updated":"2022-01-13 05:43:54.000000000","message":"ditto","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"198508f633754411de2af81f8a396855b8f38ab8","unresolved":false,"context_lines":[{"line_number":91,"context_line":"                        \u0027expected_task_state\u0027: expected_task_state,"},{"line_number":92,"context_line":"                        \u0027timeout\u0027: timeout})"},{"line_number":93,"context_line":"            if request_id:"},{"line_number":94,"context_line":"                message +\u003d \u0027 Original request ID: %s.\u0027 % request_id"},{"line_number":95,"context_line":"            message +\u003d \u0027 Current status: %s.\u0027 % server_status"},{"line_number":96,"context_line":"            message +\u003d \u0027 Current task state: %s.\u0027 % task_state"},{"line_number":97,"context_line":"            caller \u003d test_utils.find_test_caller()"}],"source_content_type":"text/x-python","patch_set":3,"id":"eb996cc5_a8725f21","line":94,"range":{"start_line":94,"start_character":29,"end_line":94,"end_character":38},"in_reply_to":"873b6aad_607e0874","updated":"2022-01-13 21:12:26.000000000","message":"Done","commit_id":"42187d65a8193e42f92dffb22b792e99cf3ad691"}]}