)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"40437b5bee69bad7109b34241521d7fd38d3d215","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use oslo_config new type PortOpt for port options"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The oslo_config library provides new type PortOpt to validate the"},{"line_number":10,"context_line":"range of port now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_64bd28d7","line":9,"updated":"2015-10-22 13:06:56.000000000","message":"what does it give us?","commit_id":"fdbe4891260127e08f1b6120a9a0757a51a326d5"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"f95786e1a7c10394bb196d6c3bed9c2b985aaf8a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use oslo_config new type PortOpt for port options"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The oslo_config library provides new type PortOpt to validate the"},{"line_number":10,"context_line":"range of port now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_bcad7aea","line":9,"updated":"2015-10-23 03:42:15.000000000","message":"yeah, surprisingly we don\u0027t seem to be validating these now so we are depending on an ugly error down the line to catch this. This will at least give a nice error to the user right on startup.","commit_id":"fdbe4891260127e08f1b6120a9a0757a51a326d5"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"ef19425b8e18dba7dc5b61b3693efcb08b0125e4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use oslo_config new type PortOpt for port options"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The oslo_config library provides new type PortOpt to validate the"},{"line_number":10,"context_line":"range of port now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_5f9b0318","line":9,"in_reply_to":"3a29b11f_4466da2a","updated":"2015-10-22 20:12:14.000000000","message":"Well I thought it was obvious that it checks that the port number is in the range 1..65535, and I looked in oslo-config and indeed that is what it does.","commit_id":"fdbe4891260127e08f1b6120a9a0757a51a326d5"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"4841dcebc2e78f7963fe515380dbe493e062aeb8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use oslo_config new type PortOpt for port options"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The oslo_config library provides new type PortOpt to validate the"},{"line_number":10,"context_line":"range of port now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_6a95df30","line":9,"in_reply_to":"3a29b11f_5f9b0318","updated":"2015-10-22 20:39:33.000000000","message":"ok thanks henry","commit_id":"fdbe4891260127e08f1b6120a9a0757a51a326d5"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"4f017c29fcd7c15bc57a87a5048b000ef555a33e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use oslo_config new type PortOpt for port options"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The oslo_config library provides new type PortOpt to validate the"},{"line_number":10,"context_line":"range of port now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_4466da2a","line":9,"in_reply_to":"3a29b11f_64bd28d7","updated":"2015-10-22 19:42:29.000000000","message":"+1 pls explain the difference or benefit from using it.","commit_id":"fdbe4891260127e08f1b6120a9a0757a51a326d5"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"f66b1d7ddccac68705fca315fe40a5d15222b82b","unresolved":false,"context_lines":[{"line_number":10,"context_line":"range of port now."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2"},{"line_number":13,"context_line":"Depends-On: Ida294b05a85f5bef587b761fcd03c28c7a3474d8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_2aa7e70b","line":13,"updated":"2015-10-22 20:21:06.000000000","message":"Actually, this dependency is not enough. That patch has to merge, then the proposal bot must create a patch for neutron, and you must depend on that.","commit_id":"fdbe4891260127e08f1b6120a9a0757a51a326d5"}]}