yahoo-eng-team team mailing list archive

[Pavlukhin Max <1739411@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

Environment: Multi-Node Lab DevStack stable/ocata

Steps to introduce:
1) Create QoS policy with dscp mark
2) Apply it to port/network

Summary: there is no dscp mark

Logs (and git diff to show patch):
http://paste.openstack.org/show/629451/


Why I didn't upload the patch?
Neutron stable/ocata version has a git repository. I pulled the newest commit, made build and I have this bug (DSCP mark disappear). Then I resolved this issue. But after that I saw the following:

There were two patches to resolve this issue but this changes weren't merged with stable/ocata branch. And there are two opinions of fixing this bug:
1) Keep code in stable/ocata unchanged and don't fix this bug (use delete_flows functions)
https://review.openstack.org/#/c/482569/2/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py
2) Change delete_flow function to uninstall_flows and fix this bug
https://review.openstack.org/#/c/406731/7/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py

By the way in pike and in master we have the second option. I prefer the
second too.

** Affects: neutron
     Importance: Undecided
         Status: Confirmed


** Tags: qos

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

Title:
  QoS DSCP mark disappear stable/ocata

Status in neutron:
  Confirmed

Bug description:
  Environment: Multi-Node Lab DevStack stable/ocata

  Steps to introduce:
  1) Create QoS policy with dscp mark
  2) Apply it to port/network

  Summary: there is no dscp mark

  Logs (and git diff to show patch):
  http://paste.openstack.org/show/629451/

  
  Why I didn't upload the patch?
  Neutron stable/ocata version has a git repository. I pulled the newest commit, made build and I have this bug (DSCP mark disappear). Then I resolved this issue. But after that I saw the following:

  There were two patches to resolve this issue but this changes weren't merged with stable/ocata branch. And there are two opinions of fixing this bug:
  1) Keep code in stable/ocata unchanged and don't fix this bug (use delete_flows functions)
  https://review.openstack.org/#/c/482569/2/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py
  2) Change delete_flow function to uninstall_flows and fix this bug
  https://review.openstack.org/#/c/406731/7/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py

  By the way in pike and in master we have the second option. I prefer
  the second too.

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