yahoo-eng-team team mailing list archive

[Rodolfo Alonso <rodolfo.alonso.hernandez@xxxxxxxxx>]

Public bug reported:

Now https://bugs.launchpad.net/neutron/+bug/1586056 is implemented and
merged, the check made in QoS agent extension when handling the QoS
rules is not needed.

In [1] and [2], a check was made to test if the set of rules in the QoS
policy was supported by the backend. Now this check is not needed
because during the assignment of a QoS policy to a port, this check is
made. The agent will handle only rules supported by the driver.

[1] https://github.com/openstack/neutron/blob/master/neutron/agent/l2/extensions/qos.py#L93
[2] https://github.com/openstack/neutron/blob/master/neutron/agent/l2/extensions/qos.py#L114

** Affects: neutron
     Importance: Undecided
     Assignee: Rodolfo Alonso (rodolfo-alonso-hernandez)
         Status: New


** Tags: qos

** Tags added: qos

** Changed in: neutron
     Assignee: (unassigned) => Rodolfo Alonso (rodolfo-alonso-hernandez)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1686627

Title:
  Remove QoS rule check in agent

Status in neutron:
  New

Bug description:
  Now https://bugs.launchpad.net/neutron/+bug/1586056 is implemented and
  merged, the check made in QoS agent extension when handling the QoS
  rules is not needed.

  In [1] and [2], a check was made to test if the set of rules in the
  QoS policy was supported by the backend. Now this check is not needed
  because during the assignment of a QoS policy to a port, this check is
  made. The agent will handle only rules supported by the driver.

  [1] https://github.com/openstack/neutron/blob/master/neutron/agent/l2/extensions/qos.py#L93
  [2] https://github.com/openstack/neutron/blob/master/neutron/agent/l2/extensions/qos.py#L114

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