)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"6935222f29f39692631c5e7baa9880704805d002","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Implement the following fixes for the address set support:"},{"line_number":10,"context_line":"- Use an OVN transaction when creating and deleting an"},{"line_number":11,"context_line":"  address set."},{"line_number":12,"context_line":"- Document the address set name requirements."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifd3e28d667b6aa5123a9477fd3f317ef11085d5b"},{"line_number":15,"context_line":"Related-Bug: #1560817"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_9a2c9e03","line":12,"updated":"2016-07-26 13:28:13.000000000","message":"Sorry for the nit of the commit message, but some comments on what would make it more clear ...\n\nThe first line is a bit too broad and doesn\u0027t really explain what is being fixed.  It suggests that perhaps this should have been multiple commits.  In this case, I think one commit is fine since the second item (doc fix) is probably too trivial to split out.  I would rewrite this as something like:\n\n    Use OVSDB transactions for address sets.\n\n    This patch ensures that a proper OVSDB transaction is\n    used when creating and deleting address sets.\n\n    In passing, also update a comment clarifying OVN\n    address set naming requirements.","commit_id":"d3ccef4f837aeb688f6e9b8d9c976a1ecb29d04b"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"5dd79ff75868483fc2ff592a20e7dc42d6d4a898","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Implement the following fixes for the address set support:"},{"line_number":10,"context_line":"- Use an OVN transaction when creating and deleting an"},{"line_number":11,"context_line":"  address set."},{"line_number":12,"context_line":"- Document the address set name requirements."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifd3e28d667b6aa5123a9477fd3f317ef11085d5b"},{"line_number":15,"context_line":"Related-Bug: #1560817"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_3a53f23a","line":12,"in_reply_to":"dada55a8_9a2c9e03","updated":"2016-07-26 13:32:22.000000000","message":"Thanks Russell.  I will update the commit message.","commit_id":"d3ccef4f837aeb688f6e9b8d9c976a1ecb29d04b"}]}
