yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #82758
[Bug 1874282] Re: [OVN] extra_dhcp_opts does not validate any options
Reviewed: https://review.opendev.org/722025
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=f6010f60429347485464107b3a838065f2f7ff7d
Submitter: Zuul
Branch: master
commit f6010f60429347485464107b3a838065f2f7ff7d
Author: Lucas Alvares Gomes <lucasagomes@xxxxxxxxx>
Date: Wed Apr 22 15:59:33 2020 +0100
[OVN] Enhance port's extra DHCP options support
Prior to this patch OVN did not validate any extra DHCP option passed
to the port leading to confusion because the user of the API could just
input any value and OVN would accept it (returning 200) but ignoring the
option internally.
This patch now adds such validations on port creation and update.
This patch also sync with the latest supported DHCP options from OVN and
create a map between the different names and option codes to their OVN
counterpart.
Closes-bug: #1874282
Change-Id: I99799e54e941cdd8da2614feecad1ef6299703fc
Signed-off-by: Lucas Alvares Gomes <lucasagomes@xxxxxxxxx>
** Changed in: neutron
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1874282
Title:
[OVN] extra_dhcp_opts does not validate any options
Status in neutron:
Fix Released
Bug description:
The OVN driver supports the extra_dhcp_opts extension, however, it
does not validate any options passed to it leading to a lot of
confusion. The user can input any value and OVN will accept it via the
API (retuning 200) but, internally it will just ignore that option if
not supported.
E.g:
(overcloud) [stack@undercloud-0 ~]$ neutron port-update --extra-dhcp-opt opt_name='interface-mtu',opt_value='1200' test_port
neutron CLI is deprecated and will be removed in the future. Use openstack CLI instead.
Updated port: test_port
(overcloud) [stack@undercloud-0 ~]$ openstack port show test_port | grep -i dhcp
| extra_dhcp_opts | ip_version='4', opt_name='interface-mtu', opt_value='1200'
()[root@controller-2 /]# ovn-nbctl list dhcp_options a825bbdb-3de2-4cd2-aa75-11951894c087
_uuid : a825bbdb-3de2-4cd2-aa75-11951894c087
cidr : "192.168.30.0/24"
external_ids : {"neutron:revision_number"="0", subnet_id="c35226d1-0984-4887-ad38-750c04fba350"}
options : {classless_static_route="{169.254.169.254/32,192.168.30.2, 0.0.0.0/0,192.168.30.1}", dns_server="{172.16.0.1, 10.0.0.1}", lease_time="43200", mtu="1442", router="192.168.30.1", server_id="192.168.30.1", server_mac="fa:16:3e:95:ec:6f"}
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1874282/+subscriptions
References