Activity log for bug #1814882

Date Who What changed Old value New value Message
2019-02-06 12:01:12 Stephen Finucane bug added bug
2019-02-06 12:02:23 Stephen Finucane description Libvirt supports setting bandwidth limits for various VIF types. https://github.com/openstack/nova/blob/bcfd2439bab7cfad942d7e6a187df6edb1d1bf09/nova/virt/libvirt/vif.py#L576 This is supported by pretty much all VIF types except one: generic vHost user interfaces. This is a mistake and should be corrected. Libvirt supports setting bandwidth limits for various VIF types.   https://github.com/openstack/nova/blob/bcfd2439bab7cfad942d7e6a187df6edb1d1bf09/nova/virt/libvirt/vif.py#L576 This is supported by pretty much all VIF types including vHost User interfaces defined by os-vif. However, generic vHost user interfaces do not set this field. This is a mistake and should be corrected.
2019-02-06 13:33:37 Matt Riedemann tags libvirt
2019-02-06 13:33:45 Matt Riedemann nova: status New Confirmed
2019-02-06 13:33:48 Matt Riedemann nova: importance Undecided Medium
2019-02-06 14:41:29 OpenStack Infra nova: status Confirmed In Progress
2019-02-06 14:41:29 OpenStack Infra nova: assignee Stephen Finucane (stephenfinucane)
2019-02-12 16:54:33 Stephen Finucane description Libvirt supports setting bandwidth limits for various VIF types.   https://github.com/openstack/nova/blob/bcfd2439bab7cfad942d7e6a187df6edb1d1bf09/nova/virt/libvirt/vif.py#L576 This is supported by pretty much all VIF types including vHost User interfaces defined by os-vif. However, generic vHost user interfaces do not set this field. This is a mistake and should be corrected. Libvirt supports setting bandwidth limits for various VIF types.   https://github.com/openstack/nova/blob/bcfd2439bab7cfad942d7e6a187df6edb1d1bf09/nova/virt/libvirt/vif.py#L576 This is supported by pretty much all VIF types including vHost User interfaces defined by os-vif. However, generic vHost user interfaces do not set this field. This is a mistake and should be corrected. UPDATE: It appears the opposite is true. We're setting this for vhostuser interfaces but shouldn't be. See discussion starting at [1]. [1] http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-02-06.log.html#t2019-02-06T16:28:10
2019-02-14 17:18:08 OpenStack Infra nova: status In Progress Fix Released