QoS policies with minimum-bandwidth rule should be rejected on non-physnet ports/networks
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
neutron |
Fix Released
|
Undecided
|
Slawek Kaplonski |
Bug Description
We seem to have forgot to reject some API operations that are actually not supported (and weren't planned to be supported) by the Stein implementation of the Guaranteed Minimum Bandwidth feature.
That is QoS policies with a minimum-bandwidth rule should not be used on ports/networks that are not backed by a physnet. But currently we allow this:
$ openstack network show private | egrep provider
| provider:
| provider:
| provider:
$ openstack network qos policy create policy0
$ openstack network qos rule create policy0 --type minimum-bandwidth --min-kbps 1000 --egress
$ openstack port create port0 --network private --qos-policy policy0
The port-create seems to work today, but on non-physnet networks there's no guarantee at all (as planned in the blueprint). Therefore I think API operations like these should be rejected now, otherwise we may set up false expectations in our users.
Changed in neutron: | |
milestone: | none → stein-rc1 |
Changed in neutron: | |
assignee: | Bence Romsics (bence-romsics) → Lajos Katona (lajos-katona) |
Changed in neutron: | |
assignee: | Lajos Katona (lajos-katona) → Bence Romsics (bence-romsics) |
Changed in neutron: | |
assignee: | Bence Romsics (bence-romsics) → Lajos Katona (lajos-katona) |
Changed in neutron: | |
assignee: | Lajos Katona (lajos-katona) → Bence Romsics (bence-romsics) |
Changed in neutron: | |
assignee: | Bence Romsics (bence-romsics) → Akihiro Motoki (amotoki) |
Changed in neutron: | |
assignee: | Akihiro Motoki (amotoki) → Slawek Kaplonski (slaweq) |
tags: | added: neutron-proactive-backport-potential |
tags: | removed: neutron-proactive-backport-potential stein-rc-potential |
tags: | added: neutron-proactive-backport-potential |
tags: | removed: neutron-proactive-backport-potential |
Fix proposed to branch: master /review. openstack. org/641712
Review: https:/