)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"84722bceb6c4d670c96e6834e6dde63be076de7a","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This also makes OVN behave like ML2/OVS, which will"},{"line_number":17,"context_line":"re-create the DHCP port when the enable_dhcp flag is"},{"line_number":18,"context_line":"changed to false and back to true."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"I am only marking this a partial fix since it is"},{"line_number":21,"context_line":"unclear if there is another issue lurking in this code."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1e04a92f_8ca3ff4e","line":18,"updated":"2023-04-11 10:43:12.000000000","message":"By the way in ml2/ovs, I believe the dhcp-agent triggers the dhcp port recreation, probably here:\nhttps://opendev.org/openstack/neutron/src/commit/a06fe7cfd9863230d29461ae8587a586f1c6d159/neutron/agent/linux/dhcp.py#L1711","commit_id":"127cde027084431cbde55b34e8442c0bf975238b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"caf56998622b677f4020f058d6c6f4ad93041222","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This also makes OVN behave like ML2/OVS, which will"},{"line_number":17,"context_line":"re-create the DHCP port when the enable_dhcp flag is"},{"line_number":18,"context_line":"changed to false and back to true."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"I am only marking this a partial fix since it is"},{"line_number":21,"context_line":"unclear if there is another issue lurking in this code."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2a3f0a07_85ad0c94","line":18,"in_reply_to":"1e04a92f_8ca3ff4e","updated":"2023-04-12 01:02:52.000000000","message":"Right, the DB code doesn\u0027t create it, but in OVN it does. This seemed like the logical place to put it based on the code (to me).","commit_id":"127cde027084431cbde55b34e8442c0bf975238b"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"222748b2cc812835b94c716be8a8f1c19e70cd26","unresolved":false,"context_lines":[{"line_number":17,"context_line":"re-create the DHCP port when the enable_dhcp flag is"},{"line_number":18,"context_line":"changed to false and back to true."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"I am only marking this a partial fix since it is"},{"line_number":21,"context_line":"unclear if there is another issue lurking in this code."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I943f2fb4db9dc33dc372f844d6133faff415befe"},{"line_number":24,"context_line":"Partial-bug: #2015377"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"5547c637_07cb77ba","line":21,"range":{"start_line":20,"start_character":1,"end_line":21,"end_character":55},"updated":"2023-04-13 15:33:30.000000000","message":"I think this patch fully addresses the issue reported","commit_id":"37a2ead37ad52b0c1f4a1dab42cfd4f8791b68d7"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8bac7c66deecc94b4203b06d1b61b9fdc5c8c223","unresolved":false,"context_lines":[{"line_number":17,"context_line":"re-create the DHCP port when the enable_dhcp flag is"},{"line_number":18,"context_line":"changed to false and back to true."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"I am only marking this a partial fix since it is"},{"line_number":21,"context_line":"unclear if there is another issue lurking in this code."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I943f2fb4db9dc33dc372f844d6133faff415befe"},{"line_number":24,"context_line":"Partial-bug: #2015377"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3a3ddfd0_25465618","line":21,"range":{"start_line":20,"start_character":1,"end_line":21,"end_character":55},"in_reply_to":"5547c637_07cb77ba","updated":"2023-04-13 16:32:55.000000000","message":"Ack, changed.","commit_id":"37a2ead37ad52b0c1f4a1dab42cfd4f8791b68d7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"708a3a6680d5d3949ff2115eb367fbb24d586ab3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"657f787b_052b945d","updated":"2023-04-07 21:23:14.000000000","message":"Going to mark this WIP for now, since it\u0027s unclear if I can add a functional test and the weekend is here.","commit_id":"127cde027084431cbde55b34e8442c0bf975238b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"caf56998622b677f4020f058d6c6f4ad93041222","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f8bc11f3_64777115","updated":"2023-04-12 01:02:52.000000000","message":"We\u0027ll see if that functional test works...","commit_id":"bc69ca0ba2415a6f6f8e55a62a66f229723d50ba"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"31cc91b5a2ae41762d699f2ff95ce48aafb59063","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"eaffb719_5d48f1b4","updated":"2023-04-12 22:47:51.000000000","message":"This is what I get for not running functional tests locally...","commit_id":"9d721d5aaa064beeedcffac84eec969accf9628d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"222748b2cc812835b94c716be8a8f1c19e70cd26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"31a0c0f1_5503c9f6","updated":"2023-04-13 15:33:30.000000000","message":"Looks good, we should maybe write a release note because that changes a bit the current behaviour.","commit_id":"37a2ead37ad52b0c1f4a1dab42cfd4f8791b68d7"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8bac7c66deecc94b4203b06d1b61b9fdc5c8c223","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e0531417_9f04b88c","in_reply_to":"31a0c0f1_5503c9f6","updated":"2023-04-13 16:32:55.000000000","message":"Ack, done.","commit_id":"37a2ead37ad52b0c1f4a1dab42cfd4f8791b68d7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6be62892ad92845473de84855e06ccccd5a2f49b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"69b7935d_43f76643","updated":"2023-04-13 16:48:20.000000000","message":"We should backport this patch","commit_id":"267efd298479d66c64d55a76bd21c9664080f76a"}]}
