)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"ff15a15e5b997c99e4de93bad254f8005839ec58","unresolved":false,"context_lines":[{"line_number":18,"context_line":"check for the overlaps in all cases."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This patch summarily removes this overlap check."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I5498c4a72b31267644da10a54a9860c1fc3bb250"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7adec928_6f4e6e98","line":21,"updated":"2014-05-21 01:21:30.000000000","message":"Can you file this as a bug on launchpad and then reference it here?\n\ne.g. Closes-Bug: #\u003cbugnumber\u003e\n\nThat exposes the patch to a larger audience (bug subscribers) that might be interested in this behavior change.","commit_id":"5ffce5decd3549c2dc1ff4aacc40700d3adea7fa"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"caf9a0f9f30524b08a7d9d32337c4867d818da2f","unresolved":false,"context_lines":[{"line_number":12,"context_line":"effect. Further, such a check actually makes it hard to use this"},{"line_number":13,"context_line":"API. For example, if a fixed IP 10.10.1.1 exists on a port and we"},{"line_number":14,"context_line":"want to allow addresses in 10.10.1.0/24 cidr on that port, then one"},{"line_number":15,"context_line":"has to configure a list of 8 cidrs ([10.10.1.0/32, 10.10.1.2/31,"},{"line_number":16,"context_line":"10.10.1.4/30, ..., 10.10.1.128/25]) on the allowed-addresspairs."},{"line_number":17,"context_line":"In addition to the above reasons, the current code also does not"},{"line_number":18,"context_line":"check for the overlaps in all cases."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7adec928_4b02233f","line":15,"updated":"2014-05-23 20:00:22.000000000","message":"I think there is some confusion here as this is not really what is required (nor the intention). That said, i\u0027m fine with this change since we don\u0027t correctly enforce this today in the update case so we might as well remove this check.","commit_id":"52301e4727091f867c42b18b316d4c4aacffea31"}]}