[OVN] QoS rules do not apply to distributed floating IP traffic

Bug #1915723 reported by Rodolfo Alonso
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Rodolfo Alonso

Bug Description

When QoS floating IP is used, the OVN QoS rules do not apply to distributed floating IP ("enable_distributed_floating_ip=True"). The condition applied in [1]:
  is_chassis_resident(chassisredirect_lrp_xxx)

is valid for non distributed floating IP deployments. The chassisredirect lrp is present in the chassis where the floating IP NAT is done.

But in distributed floating IP deployments, the NAT is done locally in the compute node that hosts the private IP port; that means the match condition should be:
  is_chassis_resident(private_port)

[1]https://github.com/openstack/neutron/blob/4a021306ad53c588dd9f8269124e7b0d49a2c752/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/extensions/qos.py#L104-L106

Tags: ovn
Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
tags: added: ovn
Changed in neutron:
importance: Undecided → Medium
Revision history for this message
Oleg Bondarev (obondarev) wrote :
Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 18.0.0.0rc1

This issue was fixed in the openstack/neutron 18.0.0.0rc1 release candidate.

Revision history for this message
Erlon R. Cruz (sombrafam) wrote :

I'm curious to know why would be the utility of having the QoS rules applied to the FIPs,
given IFIC, OVN does not support QoS at all see[1].

[1] https://docs.openstack.org/neutron/latest/ovn/gaps.html

Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

Hello Erlon:

This document is not talking about core OVN QoS support but the implementation of OVN mech driver in Neutron and the QoS support Neutron provides for this driver.

OVN FIP (max bw) is currently fully supported using the OVN QoS rules. GW IP QoS support is still under development.

Patch changing the documentation: https://review.opendev.org/c/openstack/neutron/+/777571
GW IP QoS support: https://review.opendev.org/c/openstack/neutron/+/749012

Regards.

Changed in neutron:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.