← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1843165] Re: RFE: Adding support for direct ports with qos in ovs

 

Reviewed:  https://review.opendev.org/611605
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=12089a526e25cfe5ccb8b4da86b653970d75f14a
Submitter: Zuul
Branch:    master

commit 12089a526e25cfe5ccb8b4da86b653970d75f14a
Author: waleed mousa <waleedm@xxxxxxxxxxxx>
Date:   Thu Oct 18 09:41:38 2018 -0400

    Add support for direct ports with QoS in OVS
    
    Today OVS mechanism driver can bind Direct port see [1] for OVS hardware
    offloads.
    OVS was extended with tc-offload to support rate limit see [2].
    The OVS QoS driver [3] is limited to work only with Normal Ports, so we
    can't put QoS rules on direct port.
    This patch proposes to add support in OVS QoS driver for direct ports.
    The mechanism to enforce such policies is the same with normal and
    hardware offloaded direct ports.
    
    [1] - https://github.com/openvswitch/ovs/commit/e7f6ba220e10c0b560da097185514b6e33e2dc71
    [2] - https://github.com/openstack/neutron/blob/3b074128ca91ba335bd8ef9db8dfe00fda51a63d/neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py#L83
    [3] - https://github.com/openstack/neutron/blob/3b074128ca91ba335bd8ef9db8dfe00fda51a63d/neutron/services/qos/drivers/openvswitch/driver.py#L56
    
    Change-Id: I24b5cd6c022e479080fc84e4c445c9cddfc88e38
    Closes-Bug: #1843165


** 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/1843165

Title:
  RFE: Adding support for direct ports with qos in ovs

Status in neutron:
  Fix Released

Bug description:
  Ovs mechanism driver supports direct ports for ovs hardware offlaoding
  so we need to extend the capabilities to qos as well

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1843165/+subscriptions


References