)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8d39c809c924b0caf3224c6f5218f340d9b82ea9","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2018-02-28 10:19:37 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update port profile only if defined"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ifada5a357d9a09ade7737c5fa8af1422b9353924"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f9dbf25_c50eb2e4","line":8,"updated":"2018-02-28 23:52:40.000000000","message":"Please write the reason why we need this change in the commit message.\n\nI cannot understand why we need to take care of profile only without vnic_type in this case.","commit_id":"a875cb6fbebebd7ad0fa02e7ef628e19e1ad82e5"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"3ea0d264defada3ec2c8ad3a25819b51a107a275","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update port profile only if defined"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change[1] added port profile configuration option."},{"line_number":10,"context_line":"We need to update port only if needed."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"[1] https://review.openstack.org/#/c/499575"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df7087c5_ba046919","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":38},"updated":"2018-03-14 03:10:59.000000000","message":"Can you put more details, why? What will happen if we pass {}?","commit_id":"fe3a03ff6326872cfe965575a531e10f642525ae"},{"author":{"_account_id":14807,"name":"Lenny","email":"lennyb@mellanox.com","username":"lennyb"},"change_message_id":"489b5a94b80c573b17c91c7f1dfe6b04a36f81ec","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update port profile only if defined"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change[1] added port profile configuration option."},{"line_number":10,"context_line":"We need to update port only if needed."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"[1] https://review.openstack.org/#/c/499575"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df7087c5_5c42b6ac","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":38},"in_reply_to":"df7087c5_ba046919","updated":"2018-03-15 16:25:38.000000000","message":"setting profile (including profile\u003d{}) requires Admin permissions. So no need to require Admin, unless user explicitly asked to define profile.","commit_id":"fe3a03ff6326872cfe965575a531e10f642525ae"}]}