Bug 1271094 - [Host QoS] - Updating second network with host QoS when it attached to host NIC with another network that is out-of-sync, considered as synced
Summary: [Host QoS] - Updating second network with host QoS when it attached to host N...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Network
Version: 3.6.0
Hardware: x86_64
OS: Linux
high
high
Target Milestone: ovirt-4.0.0-beta
: 4.0.0
Assignee: Martin Mucha
QA Contact: Michael Burman
URL:
Whiteboard:
Depends On:
Blocks: 1255474
TreeView+ depends on / blocked
 
Reported: 2015-10-13 07:28 UTC by Michael Burman
Modified: 2016-08-04 14:37 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Cause: invalid code Consequence: network incorrectly marked as in-sync Fix: Result: error is fixed, two (or more) networks can now be on same nic and correctly reported as in-sync or out-of-sync
Clone Of:
Environment:
Last Closed: 2016-07-05 07:58:06 UTC
oVirt Team: Network
Embargoed:
rule-engine: ovirt-4.0.0+
rule-engine: planning_ack+
danken: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1364145 0 unspecified CLOSED Forbid mixing QoS and non-QoS on the same NIC 2021-02-22 00:41:40 UTC
oVirt gerrit 56470 0 master MERGED engine: allow QoS and non-QoS networks to be attached to a nic 2016-04-26 13:05:11 UTC

Internal Links: 1364145

Description Michael Burman 2015-10-13 07:28:35 UTC
Description of problem:
[Host QoS] - Updating second network with host QoS when it attached to host NIC with another network that is out-of-sync, considered as synced.

If i have 2 networks attached to 1 NIC on host, 2 vlan networks, or 1 non-VM and 1 vlan network.
I'm updating 1 network with some host QoS via Networks tab, and this network marked now as out-of-sync.   
CanDoAction of action 'SetupNetworks' failed for user admin@internal. Reas
ons: VAR__ACTION__SETUP,VAR__TYPE__NETWORKS,ACTION_TYPE_FAILED_HOST_NETWORK_QOS_INTERFACES_WITHOUT_QOS,$ACTION_TYPE_FAILED_HOST_NETWORK_QOS_INTERFACES_WITHOUT_QOS_LIST    enp6s0,$ACTION_TYPE_FAILED_HOST_NETWORK_QO
S_INTERFACES_WITHOUT_QOS_LIST_COUNTER 1

Now, i'm updating the second network with host QoS as well and expecting it to be marked out-of-sync as well, but it marked as synced and report the host QoS parameters.
Engine thinks that the first network is synced and has host QoS parameters applied on it.

Version-Release number of selected component (if applicable):
3.6.0-0.18.el6

How reproducible:
100

Steps to Reproduce:
1. Create host QoS via DC. Create 2 networks(2 vlan networks, or 1 non-VM and 1 vlan network) and attach to 1 NIC on host via setup networks.
2. Update first network with host QoS via Networks tab
3. Update second network with host QoS via setup networks tab 

Actual results:
First network marked as out-of-sync
Second network marked as synced.

Expected results:
Both networks should be marked as out-of-sync

Comment 1 Michael Burman 2015-10-20 07:54:12 UTC
After further discussing with danken, we need to decide what will be the expected result for such scenario :

1) Both networks should be marked as out-of-sync and user will 'Sync All Networks' 
2) Once updating the second network with QoS as well, both will be considered as synced
3) Remove engine restriction for :
'All or none of the networks attached to an interface must have QoS configured'
If this will be removed, then we won't have this issue.
The first network will be considered as synced once updated with QoS and second network will be considered as synced once updated with QoS.

Comment 2 Dan Kenigsberg 2016-03-27 13:51:02 UTC
We've opted for option 3 - remove the Engine-side requirement for all-or-nothing QoS.

Comment 3 Michael Burman 2016-05-26 05:28:07 UTC
Verified on - 4.1.0-0.0.master.20160525231312.git1469b76.el7.cento and vdsm-4.18.0-50.gitd83fbc2.el7.centos.x86_64

- Engine-side requirement for all-or-nothing QoS has been removed.

Comment 4 Sandro Bonazzola 2016-07-05 07:58:06 UTC
oVirt 4.0.0 has been released, closing current release.


Note You need to log in before you can comment on or make changes to this bug.