)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28415,"name":"Lin Shuicheng","email":"shuicheng.lin@intel.com","username":"slin14"},"change_message_id":"31d81e4ce7e490cac11b73b37ae76b4163613e85","unresolved":false,"context_lines":[{"line_number":18,"context_line":"4.microcode_ctl-2.1-47.el7_6.x86_64.rpm --\u003e"},{"line_number":19,"context_line":"  microcode_ctl-2.1-47.2.el7_6.x86_64.rpm"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: 1830487"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I9c69ca78dc046128521d2a46b520f9c242fe6e56"},{"line_number":24,"context_line":"Signed-off-by: zhiguo.zhang \u003czhiguox.zhang@intel.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_542ebfa5","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":19},"updated":"2019-06-05 00:56:28.000000000","message":"Not sure why the LP is not updated with the patch info. Please help remove the blank line below, to make \"Closes-Bug\" is the last line.","commit_id":"9c9f5fc02efd8f47e74ae103867332aa8a0a4ba7"},{"author":{"_account_id":28652,"name":"Jim Somerville","email":"jim.somerville@windriver.com","username":"jsomervi"},"change_message_id":"e3e4ff17ebf32ea735861a9aa7f05325a27b29ba","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     zhiguo.zhang \u003czhiguox.zhang@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-06-05 21:17:37 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"rebase kernel patch to CentOS7.6 3.10.0-957.12.2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"1.kernel-3.10.0-957.1.3.el7.src.rpm --\u003e"},{"line_number":10,"context_line":"  kernel-3.10.0-957.12.2.el7.src.rpm"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fb8cfa7_44e28403","line":7,"updated":"2019-06-05 14:30:33.000000000","message":"The phrase \"rebase kernel patch\" is confusing, instead say \"Upgrade kernel packages to CentOS7.6 3.10.0-957.12.2\"","commit_id":"f36266a661d5b1bef9f0d41e827cc68ae82130db"},{"author":{"_account_id":28652,"name":"Jim Somerville","email":"jim.somerville@windriver.com","username":"jsomervi"},"change_message_id":"e3e4ff17ebf32ea735861a9aa7f05325a27b29ba","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"4.microcode_ctl-2.1-47.el7_6.x86_64.rpm --\u003e"},{"line_number":19,"context_line":"  microcode_ctl-2.1-47.2.el7_6.x86_64.rpm"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: 1830487"},{"line_number":22,"context_line":"Change-Id: I9c69ca78dc046128521d2a46b520f9c242fe6e56"},{"line_number":23,"context_line":"Signed-off-by: zhiguo.zhang \u003czhiguox.zhang@intel.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fb8cfa7_6499c86c","line":20,"updated":"2019-06-05 14:30:33.000000000","message":"The above 4 points don\u0027t really add any value as \"git whatchanged\" will show this anyway.\n\nA good idea is to write something simple which explains why we\u0027re doing this in the first place.  Such as \"This kernel addresses the MDS security vulnerabilities\".  It doesn\u0027t have to be a complete changelog or anything, just the top one or two points as to why this kernel increment was deemed important enough to release in the first place.","commit_id":"f36266a661d5b1bef9f0d41e827cc68ae82130db"},{"author":{"_account_id":29281,"name":"Victor Manuel Rodriguez Bahena","email":"vm.rod25@gmail.com","username":"vm-rod25"},"change_message_id":"734c13cf0bbbfdf9b091749054a44ba6f9bd5426","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"4.microcode_ctl-2.1-47.el7_6.x86_64.rpm --\u003e"},{"line_number":19,"context_line":"  microcode_ctl-2.1-47.2.el7_6.x86_64.rpm"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: 1830487"},{"line_number":22,"context_line":"Change-Id: I9c69ca78dc046128521d2a46b520f9c242fe6e56"},{"line_number":23,"context_line":"Signed-off-by: zhiguo.zhang \u003czhiguox.zhang@intel.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fb8cfa7_f880f88d","line":20,"in_reply_to":"9fb8cfa7_6499c86c","updated":"2019-06-05 15:14:03.000000000","message":"isn\u0027t enough to point to launchpad bug with a good description of each CVE?","commit_id":"f36266a661d5b1bef9f0d41e827cc68ae82130db"},{"author":{"_account_id":29281,"name":"Victor Manuel Rodriguez Bahena","email":"vm.rod25@gmail.com","username":"vm-rod25"},"change_message_id":"7c726cb3a7f935d39ef463f4290c8ae4626d07c4","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"4.microcode_ctl-2.1-47.el7_6.x86_64.rpm --\u003e"},{"line_number":19,"context_line":"  microcode_ctl-2.1-47.2.el7_6.x86_64.rpm"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: 1830487"},{"line_number":22,"context_line":"Change-Id: I9c69ca78dc046128521d2a46b520f9c242fe6e56"},{"line_number":23,"context_line":"Signed-off-by: zhiguo.zhang \u003czhiguox.zhang@intel.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fb8cfa7_fb0232b4","line":20,"in_reply_to":"9fb8cfa7_db872e61","updated":"2019-06-05 15:28:47.000000000","message":"Ok, is a simple change in the commit message.thansk for the advice, in future patches that close bugs I will include also good description on the commit message","commit_id":"f36266a661d5b1bef9f0d41e827cc68ae82130db"},{"author":{"_account_id":28652,"name":"Jim Somerville","email":"jim.somerville@windriver.com","username":"jsomervi"},"change_message_id":"3a30c3b113014097eac5a224ea8d3d8bdd32370f","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"4.microcode_ctl-2.1-47.el7_6.x86_64.rpm --\u003e"},{"line_number":19,"context_line":"  microcode_ctl-2.1-47.2.el7_6.x86_64.rpm"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: 1830487"},{"line_number":22,"context_line":"Change-Id: I9c69ca78dc046128521d2a46b520f9c242fe6e56"},{"line_number":23,"context_line":"Signed-off-by: zhiguo.zhang \u003czhiguox.zhang@intel.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fb8cfa7_db872e61","line":20,"in_reply_to":"9fb8cfa7_f880f88d","updated":"2019-06-05 15:24:46.000000000","message":"Why not save the next person from having to look up the bug to get the basic gist of the change?  That\u0027s what commit logs are for.  Commit logs shouldn\u0027t just be dumping grounds for lists of numbers.  They can look up the bug if they want more details.  In this case a simple \"Address the MDS security issues\" would suffice for the basic gist.","commit_id":"f36266a661d5b1bef9f0d41e827cc68ae82130db"}]}
