)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e291caf6d7ebc1c5f8413fbf046deabe14f779fa","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2021-04-27 21:06:17 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Initialize global data separately and run_once in WSGI app init"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We have discovered that if an exception is raised at any point during"},{"line_number":10,"context_line":"the running of the init_application WSGI script in an apache/mod_wsgi"},{"line_number":11,"context_line":"Daemon Mode environment, it will prompt apache/mod_wsgi to re-run the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6d575bab_8b803989","line":8,"updated":"2021-04-28 10:11:25.000000000","message":"I think the best would be to have here, under the title, a note, something like:\n\nNOTE: This is a combination of two patch to avoid intermittent test\nfailure, that was introduced by the original bug fix, that was fixed\nby patch I2bd360dcc6501feea7baf02d4510b282205fc061.","commit_id":"3672796075f6f8d568b5d49892a6d49752a27694"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e291caf6d7ebc1c5f8413fbf046deabe14f779fa","unresolved":true,"context_lines":[{"line_number":42,"context_line":"Change-Id: I2bd360dcc6501feea7baf02d4510b282205fc061"},{"line_number":43,"context_line":"(cherry picked from commit 7c9edc02eda45aafbbb539b759e6b92f7aeb5ea8)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Reset global wsgi app state in unit test"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Since I2bd360dcc6501feea7baf02d4510b282205fc061 there is a global state"},{"line_number":48,"context_line":"set during the wsgi_app init making our unit test cases"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6730188e_1680e8c3","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":40},"updated":"2021-04-28 10:11:25.000000000","message":"The second patch\u0027s commit message would be better to placed before the original patch\u0027s Change-Id section in my opinion.","commit_id":"3672796075f6f8d568b5d49892a6d49752a27694"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"890a61b013b1738755a2e863321427a5aa25fe27","unresolved":true,"context_lines":[{"line_number":49,"context_line":"non-deterministic based on the order of them. This patch makes sure"},{"line_number":50,"context_line":"that the global state is reset for each test case."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Change-Id: Ie2ba746a7697a70fdbc6594f02257f78becfe962"},{"line_number":53,"context_line":"Closes-Bug: #1921098"},{"line_number":54,"context_line":"(cherry picked from commit bc2c19bb2db901af0c48d34fb15a335f4e343361)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a0fa9c08_9544b068","line":52,"updated":"2021-04-27 21:24:46.000000000","message":"I\u0027m actually not sure if I did this right in keeping this Change-Id at the bottom because it makes the gerrit web UI tie this patch to the other test fix patches when you click on the Change-Id link. If anyone knows, lmk.","commit_id":"3672796075f6f8d568b5d49892a6d49752a27694"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e291caf6d7ebc1c5f8413fbf046deabe14f779fa","unresolved":true,"context_lines":[{"line_number":49,"context_line":"non-deterministic based on the order of them. This patch makes sure"},{"line_number":50,"context_line":"that the global state is reset for each test case."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Change-Id: Ie2ba746a7697a70fdbc6594f02257f78becfe962"},{"line_number":53,"context_line":"Closes-Bug: #1921098"},{"line_number":54,"context_line":"(cherry picked from commit bc2c19bb2db901af0c48d34fb15a335f4e343361)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7d5bdc3d_e5524de8","line":52,"in_reply_to":"a0fa9c08_9544b068","updated":"2021-04-28 10:11:25.000000000","message":"A commit message can have multiple Change-Ids and always the last one is the one that counts for gerrit. However, to avoid confusions, I think it\u0027s best to keep the most relevant change-id only, and keep the \u0027cherry picked from\u0027 lines from the less relevant patch(es).","commit_id":"3672796075f6f8d568b5d49892a6d49752a27694"}]}