openstack
/
neutron
Code Issues Proposed changes

Merge "QoS DSCP use mod_flow instead of mod_flows"

This commit is contained in:
Jenkins 2016-04-07 07:41:37 +00:00 committed by Gerrit Code Review
parent 7fe6c2d07b 6b6c0421bb
commit ba7c7bc0a9
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
neutron/plugins/ml2/drivers/openvswitch/agent/extension_drivers/qos_driver.py
View File

@ -102,8 +102,8 @@ class QosOVSAgentDriver(qos.QosAgentDriver):
actions = "mod_nw_tos:" + mark + ","
actions += ','.join([act for act in acts
if "mod_nw_tos:" not in act])
self.br_int.mod_flows(reg2=0, in_port=port, table=0,
actions=actions)
self.br_int.mod_flow(reg2=0, in_port=port, table=0,
actions=actions)
def delete_dscp_marking(self, port):
port_name = port['vif_port'].port_name

5
neutron/tests/functional/agent/l2/extensions/test_ovs_agent_qos_extension.py
View File

@ -223,14 +223,19 @@ class TestOVSAgentQosExtension(OVSAgentQoSExtensionTestFramework):
port_dicts=self.create_test_ports(amount=1,
policy_id=TEST_POLICY_ID1))
self.wait_until_ports_state(self.ports, up=True)
self._assert_dscp_marking_rule_is_set(self.ports[0],
TEST_DSCP_MARKING_RULE_1)
policy_copy = copy.deepcopy(self.qos_policies[TEST_POLICY_ID1])
policy_copy.rules[0].max_kbps = 500
policy_copy.rules[0].max_burst_kbps = 5
policy_copy.rules[1].dscp_mark = TEST_DSCP_MARK_2
consumer_reg.push(resources.QOS_POLICY, policy_copy, events.UPDATED)
self.wait_until_bandwidth_limit_rule_applied(self.ports[0],
policy_copy.rules[0])
self._assert_bandwidth_limit_rule_is_set(self.ports[0],
policy_copy.rules[0])
self._assert_dscp_marking_rule_is_set(self.ports[0],
TEST_DSCP_MARKING_RULE_2)
def test_port_qos_disassociation(self):
"""Test that qos_policy_id set to None will remove all qos rules from