)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"f776e1b3e13a71acb1c476a5db9197cce9164e32","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2019-05-09 14:26:01 +0800"},{"line_number":4,"context_line":"Commit:     ZhongShengping \u003cchdzsp@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-05-09 14:26:01 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update Python 3 test runtimes for Train"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following the process set down in the TC resolution on updating Python"},{"line_number":10,"context_line":"versions:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_f3a56405","line":7,"range":{"start_line":6,"start_character":0,"end_line":7,"end_character":39},"updated":"2019-05-09 09:04:15.000000000","message":"This summary is confusing. We already have unit test entries for py36 and py37. This change actually changes nothing. It just changes how the jobs are defined in zuul. Please update the commit message more appropriately.","commit_id":"4d49fd797be3ea3c9c15afae0bd876d5a01733b7"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a08a18d3e6484516ffe4bea2b168fd32738445dc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     ZhongShengping \u003cchdzsp@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-05-09 17:16:52 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update Python 3 test runtimes for Train"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This goal is to implement the process set out in the 2018-10-24 Python"},{"line_number":10,"context_line":"Update Process TC resolution[1], for the Train cycle to ensure unit"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dfbec78f_53a550b4","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":39},"updated":"2019-05-09 09:34:39.000000000","message":"My previous comment was ignored??\n\nYou added the general information but it does not address my comment.","commit_id":"60a38df6ed65f64cac84bb0e7b338bcc43e191bf"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"ba3cab8ad781aa9de8610edc93895776be22a254","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     ZhongShengping \u003cchdzsp@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-05-09 17:16:52 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update Python 3 test runtimes for Train"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This goal is to implement the process set out in the 2018-10-24 Python"},{"line_number":10,"context_line":"Update Process TC resolution[1], for the Train cycle to ensure unit"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dfbec78f_338bfc1f","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":39},"in_reply_to":"dfbec78f_53a550b4","updated":"2019-05-09 09:36:02.000000000","message":"For example, \"Use zuul template for python3 test runtime\"?","commit_id":"60a38df6ed65f64cac84bb0e7b338bcc43e191bf"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"29f7bb6e5ba03a1e4f755f2083d47c4b9bba50db","unresolved":false,"context_lines":[{"line_number":17,"context_line":"[2]https://governance.openstack.org/tc/reference/runtimes/train.html"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I406a4f1ecbff5b6c8e18ff1b93a3b2ceef49343a"},{"line_number":20,"context_line":"Depends-On: https://review.opendev.org/#/c/641878/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dfbec78f_4469f188","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":50},"updated":"2019-05-10 07:23:41.000000000","message":"remove Depends-on as this patch is already merged.","commit_id":"a816417157fe9f6582dc1eafaae1672315923d8d"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"6defacfbb417e3e4e33b5c293171204636d13f7a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"[2]https://governance.openstack.org/tc/reference/runtimes/train.html"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I406a4f1ecbff5b6c8e18ff1b93a3b2ceef49343a"},{"line_number":20,"context_line":"Depends-On: https://review.opendev.org/#/c/641878/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dfbec78f_a428ed33","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":50},"in_reply_to":"dfbec78f_4469f188","updated":"2019-05-10 10:39:17.000000000","message":"Done.","commit_id":"a816417157fe9f6582dc1eafaae1672315923d8d"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"44293342d0364919c64e266f7552575590e9876b","unresolved":false,"context_lines":[{"line_number":17,"context_line":"[2]https://governance.openstack.org/tc/reference/runtimes/train.html"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I406a4f1ecbff5b6c8e18ff1b93a3b2ceef49343a"},{"line_number":20,"context_line":"Depends-On: https://review.opendev.org/#/c/641878/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dfbec78f_fdcaf817","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":50},"in_reply_to":"dfbec78f_4469f188","updated":"2019-05-10 10:23:13.000000000","message":"question: Is there any problem if we keep Depends-On flag?","commit_id":"a816417157fe9f6582dc1eafaae1672315923d8d"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"7c39b3da520a4d9a87dbe8c358d67ec5b5264e45","unresolved":false,"context_lines":[{"line_number":17,"context_line":"[2]https://governance.openstack.org/tc/reference/runtimes/train.html"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I406a4f1ecbff5b6c8e18ff1b93a3b2ceef49343a"},{"line_number":20,"context_line":"Depends-On: https://review.opendev.org/#/c/641878/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dfbec78f_fb00b203","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":50},"in_reply_to":"dfbec78f_a428ed33","updated":"2019-05-11 01:10:22.000000000","message":"Thanks for clarification. Generally speaking, we don\u0027t need to do so, but it sounds okay for this case.","commit_id":"a816417157fe9f6582dc1eafaae1672315923d8d"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"6defacfbb417e3e4e33b5c293171204636d13f7a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"[2]https://governance.openstack.org/tc/reference/runtimes/train.html"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I406a4f1ecbff5b6c8e18ff1b93a3b2ceef49343a"},{"line_number":20,"context_line":"Depends-On: https://review.opendev.org/#/c/641878/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dfbec78f_586a32b3","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":50},"in_reply_to":"dfbec78f_fdcaf817","updated":"2019-05-10 10:39:17.000000000","message":"The dependent patch was merged on 11th March and this patch was pushed on 9th May, so this tag wasn\u0027t required in the first place.\nIMO there is no point in keeping irrelevant tags in commit message. Although it could have been restated showing that \"due to xyz change this patch is required\" but Depends-On is a reserved tag showing cross-project dependency which i think in this case isn\u0027t necessary.","commit_id":"a816417157fe9f6582dc1eafaae1672315923d8d"}]}