qos scenario tests should be skipped if rule type is not supported

Bug #1694396 reported by YAMAMOTO Takashi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
YAMAMOTO Takashi

Bug Description

related bug 1694190

it should skip if the functionality is not supported.
it has require_qos_rule_type check but it's broken after the recent api semantics change. [1]
the problem is that if a deployment has multiple drivers and one of them doesn't support the
rule type, the test doesn't skip (anymore) and can fail.

[1] Ia00d349625db358ab486802fc0ff2e69eaa3895e

Tags: qos
Changed in neutron:
assignee: nobody → YAMAMOTO Takashi (yamamoto)
status: New → In Progress
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :
tags: added: qos
Changed in neutron:
importance: Undecided → Medium
importance: Medium → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/468982
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=55d810c7e68dd5379a55da45f1cf43ac78573497
Submitter: Jenkins
Branch: master

commit 55d810c7e68dd5379a55da45f1cf43ac78573497
Author: YAMAMOTO Takashi <email address hidden>
Date: Tue May 30 03:01:26 2017 +0000

    Revert "Change list of available qos rules"

    This reverts commit 3299cdffae5cd7196a1676da103da5e2e413ec21.

    At least, there's a user which relies on the previous
    semantics. (Our tempest plugin)
    We should not change API semantics lightly
    in an incompatible way.

    Closes-Bug: #1694396
    Related-Bug: #1694190
    Change-Id: I88a216951d8996ac8bc90078b4239f0d25392e58

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 11.0.0.0b3

This issue was fixed in the openstack/neutron 11.0.0.0b3 development milestone.

tags: added: neutron-proactive-backport-potential
tags: removed: neutron-proactive-backport-potential
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.