)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a92ae1970c3ea7984b8cd7f21394555e8d89ccf8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4ed5db6e_fa07611a","updated":"2022-05-03 14:32:20.000000000","message":"-2 until we properly test this","commit_id":"186c2ed5b09bd1b9f57db5d5168132926f21ab47"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6bcaed2e0ee02ac4cb58fd722f4e57e68abf8dfa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eda864a0_a91878d3","updated":"2022-05-06 10:40:12.000000000","message":"Will add a release note for this","commit_id":"01a8e93be3ed201999193bbcc39ed34fd134b423"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"883e25fcdceb0279dd3a43c1f0a6ac5e1440857f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"756f9725_524c2c66","updated":"2022-06-01 13:11:01.000000000","message":"@Lucas, before approving I think that the reno could possibly state the minimum core OVN version needed for this feature? Wdyt? ","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f58af3c4b295774f23dcb5ec4cf6251bddfab45a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"94a64e38_2ed38df3","updated":"2022-05-31 09:16:35.000000000","message":"All questions clarified","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"3025e49bc04e0bfccc583ace191cbfdb81407d08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"71d88a7f_c715a49f","updated":"2022-06-01 13:03:14.000000000","message":"Fantastic! Thanks a lot Lucas!","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6c8ccc6094e675a8a6fa96d03886a3c124c0c81f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0aafee80_8865475b","updated":"2022-05-30 14:12:23.000000000","message":"Some questions","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"bcc2af4ba4952ff5a1490ef1e432522e5470fc02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"19cd8e5d_1ba5f98e","updated":"2022-05-31 08:19:53.000000000","message":"Thanks Rodolfo for the reviews, I hope I have answered it all inline.","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"8b7031870ad5d60ee8efcd755312e9f86d4d8fbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4232d0fe_60e3f7cf","updated":"2022-06-02 07:58:52.000000000","message":"recheck","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"cfea0dcad865c80287bc5f2e26459568909c04db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"160397ea_2eecd58a","updated":"2022-06-02 09:45:37.000000000","message":"recheck\n\nunrelated failure in functional test test_dvr_local_router_fip_qos","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"8ed0d54c0dd2718a2a77f77a88c899583ef3ed9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3699d540_c039853b","in_reply_to":"756f9725_524c2c66","updated":"2022-06-01 13:13:28.000000000","message":"That will be good but, I plan to put it in the doc patch too. I can amend the reno in that patch if u r ok with it.","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"}],"neutron/common/ovn/constants.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6c8ccc6094e675a8a6fa96d03886a3c124c0c81f","unresolved":true,"context_lines":[{"line_number":136,"context_line":"        \u0027wpad\u0027: \u0027wpad\u0027,"},{"line_number":137,"context_line":"        \u0027path-prefix\u0027: \u0027path_prefix\u0027,"},{"line_number":138,"context_line":"        \u0027tftp-server-address\u0027: \u0027tftp_server_address\u0027,"},{"line_number":139,"context_line":"        \u0027server-ip-address\u0027: \u0027next_server\u0027,"},{"line_number":140,"context_line":"        \u00271\u0027: \u0027netmask\u0027,"},{"line_number":141,"context_line":"        \u00273\u0027: \u0027router\u0027,"},{"line_number":142,"context_line":"        \u00276\u0027: \u0027dns_server\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"e1f470e8_95f744d5","line":139,"range":{"start_line":139,"start_character":8,"end_line":139,"end_character":43},"updated":"2022-05-30 14:12:23.000000000","message":"Ok, so that was a previous mistake, before this implementation.\n\nGood catch.","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"bcc2af4ba4952ff5a1490ef1e432522e5470fc02","unresolved":true,"context_lines":[{"line_number":136,"context_line":"        \u0027wpad\u0027: \u0027wpad\u0027,"},{"line_number":137,"context_line":"        \u0027path-prefix\u0027: \u0027path_prefix\u0027,"},{"line_number":138,"context_line":"        \u0027tftp-server-address\u0027: \u0027tftp_server_address\u0027,"},{"line_number":139,"context_line":"        \u0027server-ip-address\u0027: \u0027next_server\u0027,"},{"line_number":140,"context_line":"        \u00271\u0027: \u0027netmask\u0027,"},{"line_number":141,"context_line":"        \u00273\u0027: \u0027router\u0027,"},{"line_number":142,"context_line":"        \u00276\u0027: \u0027dns_server\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"986b1738_a974d798","line":139,"range":{"start_line":139,"start_character":8,"end_line":139,"end_character":43},"in_reply_to":"e1f470e8_95f744d5","updated":"2022-05-31 08:19:53.000000000","message":"That\u0027s right, it was pointing to the wrong DHCP option. In fact, this \"tftp_server_address\" is the equivalent of option 150 which is a Cisco option [0].\n\n[0] https://blog.router-switch.com/2013/03/dhcp-option-150-dhcp-option-66/","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"}],"neutron/common/ovn/utils.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6c8ccc6094e675a8a6fa96d03886a3c124c0c81f","unresolved":true,"context_lines":[{"line_number":157,"context_line":"            failed \u003d False"},{"line_number":158,"context_line":"            break"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"        if opt_name not in mapping[ip_version]:"},{"line_number":161,"context_line":"            invalid[ip_version].append(opt_name)"},{"line_number":162,"context_line":"            failed \u003d True"},{"line_number":163,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"84188c27_0aac4d4b","line":160,"range":{"start_line":160,"start_character":27,"end_line":160,"end_character":46},"updated":"2022-05-30 14:12:23.000000000","message":"Just one question: BM DHCP IPv6 is not supported. Should we fail here? Log that?","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f58af3c4b295774f23dcb5ec4cf6251bddfab45a","unresolved":false,"context_lines":[{"line_number":157,"context_line":"            failed \u003d False"},{"line_number":158,"context_line":"            break"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"        if opt_name not in mapping[ip_version]:"},{"line_number":161,"context_line":"            invalid[ip_version].append(opt_name)"},{"line_number":162,"context_line":"            failed \u003d True"},{"line_number":163,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"6be73c1b_b485a00d","line":160,"range":{"start_line":160,"start_character":27,"end_line":160,"end_character":46},"in_reply_to":"6aa2eeb9_119ab45f","updated":"2022-05-31 09:16:35.000000000","message":"Done","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"bcc2af4ba4952ff5a1490ef1e432522e5470fc02","unresolved":true,"context_lines":[{"line_number":157,"context_line":"            failed \u003d False"},{"line_number":158,"context_line":"            break"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"        if opt_name not in mapping[ip_version]:"},{"line_number":161,"context_line":"            invalid[ip_version].append(opt_name)"},{"line_number":162,"context_line":"            failed \u003d True"},{"line_number":163,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"6aa2eeb9_119ab45f","line":160,"range":{"start_line":160,"start_character":27,"end_line":160,"end_character":46},"in_reply_to":"84188c27_0aac4d4b","updated":"2022-05-31 08:19:53.000000000","message":"It will be logged already. As the options sent by Ironic will not match the ones from SUPPORTED_DHCP_OPTS_MAPPING (remember the SUPPORTED_BM_DHCP_OPTS_MAPPING mapping is a copy of that other mapping with the specific baremetal options, dnsmasq syntax, in it). \n\nSo when IPv6 is used, our code will already detect that as a invalid option and it will be logged here: https://github.com/openstack/neutron/blob/7fc509aa9aabf2e7c519d90570d9b56031abf0ab/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L658-L668","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"}],"neutron/conf/plugins/ml2/drivers/ovn/ovn_conf.py":[{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"73b43562e51bc08db08ca2f6b2f911a779c225de","unresolved":true,"context_lines":[{"line_number":207,"context_line":"                       \u0027or by checking the output of the following command: \\n\u0027"},{"line_number":208,"context_line":"                       \u0027ovs-appctl -t ovs-vswitchd dpif/show-dp-features \u0027"},{"line_number":209,"context_line":"                       \u0027br-int | grep \"Check pkt length action\".\u0027)),"},{"line_number":210,"context_line":"    cfg.BoolOpt(\u0027disable_ovn_dhcp_for_baremetal_ports\u0027,"},{"line_number":211,"context_line":"                default\u003dFalse,"},{"line_number":212,"context_line":"                help\u003d_(\u0027Disable OVN\\\u0027s built-in DHCP for baremetal ports \u0027"},{"line_number":213,"context_line":"                       \u0027(VNIC type \"baremetal\"). This alllow operators to \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"3361cdb0_c7f0f932","line":210,"range":{"start_line":210,"start_character":17,"end_line":210,"end_character":53},"updated":"2022-05-06 11:10:04.000000000","message":"If this is changed from True (disabled) to False (enabled), we\u0027ll need to create the external ports and all at startup time, right?","commit_id":"01a8e93be3ed201999193bbcc39ed34fd134b423"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"08fc41f6894858b272c9f127739bbd2d70042537","unresolved":true,"context_lines":[{"line_number":207,"context_line":"                       \u0027or by checking the output of the following command: \\n\u0027"},{"line_number":208,"context_line":"                       \u0027ovs-appctl -t ovs-vswitchd dpif/show-dp-features \u0027"},{"line_number":209,"context_line":"                       \u0027br-int | grep \"Check pkt length action\".\u0027)),"},{"line_number":210,"context_line":"    cfg.BoolOpt(\u0027disable_ovn_dhcp_for_baremetal_ports\u0027,"},{"line_number":211,"context_line":"                default\u003dFalse,"},{"line_number":212,"context_line":"                help\u003d_(\u0027Disable OVN\\\u0027s built-in DHCP for baremetal ports \u0027"},{"line_number":213,"context_line":"                       \u0027(VNIC type \"baremetal\"). This alllow operators to \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"4b517664_d2df3c5e","line":210,"range":{"start_line":210,"start_character":17,"end_line":210,"end_character":53},"in_reply_to":"3361cdb0_c7f0f932","updated":"2022-05-06 11:33:45.000000000","message":"Good point, perhaps a maintenance task might be needed here","commit_id":"01a8e93be3ed201999193bbcc39ed34fd134b423"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6f94fbfc3e8890920c6a861e73390e634fd4289d","unresolved":false,"context_lines":[{"line_number":207,"context_line":"                       \u0027or by checking the output of the following command: \\n\u0027"},{"line_number":208,"context_line":"                       \u0027ovs-appctl -t ovs-vswitchd dpif/show-dp-features \u0027"},{"line_number":209,"context_line":"                       \u0027br-int | grep \"Check pkt length action\".\u0027)),"},{"line_number":210,"context_line":"    cfg.BoolOpt(\u0027disable_ovn_dhcp_for_baremetal_ports\u0027,"},{"line_number":211,"context_line":"                default\u003dFalse,"},{"line_number":212,"context_line":"                help\u003d_(\u0027Disable OVN\\\u0027s built-in DHCP for baremetal ports \u0027"},{"line_number":213,"context_line":"                       \u0027(VNIC type \"baremetal\"). This alllow operators to \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"67e62994_6514686d","line":210,"range":{"start_line":210,"start_character":17,"end_line":210,"end_character":53},"in_reply_to":"4b517664_d2df3c5e","updated":"2022-05-24 10:36:44.000000000","message":"Done","commit_id":"01a8e93be3ed201999193bbcc39ed34fd134b423"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6c8ccc6094e675a8a6fa96d03886a3c124c0c81f","unresolved":true,"context_lines":[{"line_number":209,"context_line":"                       \u0027br-int | grep \"Check pkt length action\".\u0027)),"},{"line_number":210,"context_line":"    cfg.BoolOpt(\u0027disable_ovn_dhcp_for_baremetal_ports\u0027,"},{"line_number":211,"context_line":"                default\u003dFalse,"},{"line_number":212,"context_line":"                help\u003d_(\u0027Disable OVN\\\u0027s built-in DHCP for baremetal ports \u0027"},{"line_number":213,"context_line":"                       \u0027(VNIC type \"baremetal\"). This alllow operators to \u0027"},{"line_number":214,"context_line":"                       \u0027plug their own DHCP server of choice for PXE booting \u0027"},{"line_number":215,"context_line":"                       \u0027baremetal nodes. Defaults to False.\u0027)),"}],"source_content_type":"text/x-python","patch_set":9,"id":"de366ad1_e0fede6b","line":212,"range":{"start_line":212,"start_character":35,"end_line":212,"end_character":36},"updated":"2022-05-30 14:12:23.000000000","message":"nit: hehehe you can use double quotes to enclose the text","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"bcc2af4ba4952ff5a1490ef1e432522e5470fc02","unresolved":true,"context_lines":[{"line_number":209,"context_line":"                       \u0027br-int | grep \"Check pkt length action\".\u0027)),"},{"line_number":210,"context_line":"    cfg.BoolOpt(\u0027disable_ovn_dhcp_for_baremetal_ports\u0027,"},{"line_number":211,"context_line":"                default\u003dFalse,"},{"line_number":212,"context_line":"                help\u003d_(\u0027Disable OVN\\\u0027s built-in DHCP for baremetal ports \u0027"},{"line_number":213,"context_line":"                       \u0027(VNIC type \"baremetal\"). This alllow operators to \u0027"},{"line_number":214,"context_line":"                       \u0027plug their own DHCP server of choice for PXE booting \u0027"},{"line_number":215,"context_line":"                       \u0027baremetal nodes. Defaults to False.\u0027)),"}],"source_content_type":"text/x-python","patch_set":9,"id":"922d3ca0_01c723a5","line":212,"range":{"start_line":212,"start_character":35,"end_line":212,"end_character":36},"in_reply_to":"de366ad1_e0fede6b","updated":"2022-05-31 08:19:53.000000000","message":"Noted, if I need to send a new patch-set I will fix this.","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"0cf8031ee44de783ced9b97f37ed55e90ab61c44","unresolved":true,"context_lines":[{"line_number":731,"context_line":"                    txn.add(cmd)"},{"line_number":732,"context_line":"        raise periodics.NeverAgain()"},{"line_number":733,"context_line":""},{"line_number":734,"context_line":"    # A static spacing value is used here, but this method will only run"},{"line_number":735,"context_line":"    # once per lock due to the use of periodics.NeverAgain()."},{"line_number":736,"context_line":"    @periodics.periodic(spacing\u003d600, run_immediately\u003dTrue)"},{"line_number":737,"context_line":"    def check_baremetal_ports_dhcp_options(self):"},{"line_number":738,"context_line":"        \"\"\"Update baremetal ports DHCP options"}],"source_content_type":"text/x-python","patch_set":9,"id":"fee8d524_c746b3ec","line":735,"range":{"start_line":734,"start_character":4,"end_line":735,"end_character":61},"updated":"2022-05-30 10:00:33.000000000","message":"thanks for explanation, I read a few times this method to understand it really","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"cc903dc601e243688af20198b87761bea1870ac4","unresolved":true,"context_lines":[{"line_number":731,"context_line":"                    txn.add(cmd)"},{"line_number":732,"context_line":"        raise periodics.NeverAgain()"},{"line_number":733,"context_line":""},{"line_number":734,"context_line":"    # A static spacing value is used here, but this method will only run"},{"line_number":735,"context_line":"    # once per lock due to the use of periodics.NeverAgain()."},{"line_number":736,"context_line":"    @periodics.periodic(spacing\u003d600, run_immediately\u003dTrue)"},{"line_number":737,"context_line":"    def check_baremetal_ports_dhcp_options(self):"},{"line_number":738,"context_line":"        \"\"\"Update baremetal ports DHCP options"}],"source_content_type":"text/x-python","patch_set":9,"id":"9e7b3fef_b52cbb80","line":735,"range":{"start_line":734,"start_character":4,"end_line":735,"end_character":61},"in_reply_to":"fee8d524_c746b3ec","updated":"2022-05-30 12:32:42.000000000","message":"Thanks for reviewing!\n\nRight yeah, this task is used to dynamically change the baremetal ports to enable and disable DHCP based on the configuration option. So it should run only once at the node that has the active lock.","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6c8ccc6094e675a8a6fa96d03886a3c124c0c81f","unresolved":true,"context_lines":[{"line_number":734,"context_line":"    # A static spacing value is used here, but this method will only run"},{"line_number":735,"context_line":"    # once per lock due to the use of periodics.NeverAgain()."},{"line_number":736,"context_line":"    @periodics.periodic(spacing\u003d600, run_immediately\u003dTrue)"},{"line_number":737,"context_line":"    def check_baremetal_ports_dhcp_options(self):"},{"line_number":738,"context_line":"        \"\"\"Update baremetal ports DHCP options"},{"line_number":739,"context_line":""},{"line_number":740,"context_line":"        Update baremetal ports DHCP options based on the"}],"source_content_type":"text/x-python","patch_set":9,"id":"c5dfd74e_171b4539","line":737,"range":{"start_line":737,"start_character":8,"end_line":737,"end_character":42},"updated":"2022-05-30 14:12:23.000000000","message":"I think we should add a note to remove this check in Z+4, right?","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"bcc2af4ba4952ff5a1490ef1e432522e5470fc02","unresolved":true,"context_lines":[{"line_number":734,"context_line":"    # A static spacing value is used here, but this method will only run"},{"line_number":735,"context_line":"    # once per lock due to the use of periodics.NeverAgain()."},{"line_number":736,"context_line":"    @periodics.periodic(spacing\u003d600, run_immediately\u003dTrue)"},{"line_number":737,"context_line":"    def check_baremetal_ports_dhcp_options(self):"},{"line_number":738,"context_line":"        \"\"\"Update baremetal ports DHCP options"},{"line_number":739,"context_line":""},{"line_number":740,"context_line":"        Update baremetal ports DHCP options based on the"}],"source_content_type":"text/x-python","patch_set":9,"id":"2d0dddbc_763682bf","line":737,"range":{"start_line":737,"start_character":8,"end_line":737,"end_character":42},"in_reply_to":"c5dfd74e_171b4539","updated":"2022-05-31 08:19:53.000000000","message":"I don\u0027t think we should remove this. Because this maintenance task is to make sure that the configuration option is being honored.\n\nIf operator disables/enables the OVN built-in DHCP for baremetal ports we need this task to change the configuration on the ports itself.\n\nUnless in the future we may think about removing this configuration option completely and always have OVN DHCP serving baremetals as the only option. Tho, from my experience working previously with Ironic I think many operators may want to plug their own DHCP server for baremetal, even Ironic does make it configurable because there are some crazy configurations that people may want have for PXE booting their hardware.","commit_id":"e73a85f3dd15aea2564a34f36261cd4c03128450"}]}