Bug 999975 - PRD35 - [RFE] Accept vlan devices identified by any name
Summary: PRD35 - [RFE] Accept vlan devices identified by any name
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 3.4.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: 3.5.0
Assignee: Alona Kaplan
QA Contact: Meni Yakove
URL:
Whiteboard: network
: 1198005 (view as bug list)
Depends On:
Blocks: 1091863 rhev3.5beta 1156165
TreeView+ depends on / blocked
 
Reported: 2013-08-22 13:32 UTC by Amador Pahim
Modified: 2019-08-15 03:41 UTC (History)
17 users (show)

Fixed In Version: vt1.3
Doc Type: Enhancement
Doc Text:
Previously if a vlan device had a non-standard name (the standard is- "dev.VLANID"), the engine could not handle and display it. This feature adds the functionality to display such vlan devices. p.s- those devices are just displayed in host->network interfaces sub tab. Setup networks operations cannot be performed on them.
Clone Of:
: 1091863 (view as bug list)
Environment:
Last Closed: 2015-02-11 17:54:27 UTC
oVirt Team: Network
Target Upstream Version:
Embargoed:
nyechiel: Triaged+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:0158 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Virtualization Manager 3.5.0 2015-02-11 22:38:50 UTC
oVirt gerrit 26605 0 master MERGED engine: adding base_interface column to vds_interface table Never
oVirt gerrit 26606 0 master MERGED engine: Read "vlanid" and "interface" from VLAN information Never
oVirt gerrit 26607 0 master MERGED engine: remove unnecessary parameters from VBOB Never
oVirt gerrit 26608 0 master MERGED engine: Adjust NetworkUtils to accept any vlan device name- phase 1 Never
oVirt gerrit 26609 0 master MERGED engine: Adjust MigrateVmCommand to accept any vlan device name Never
oVirt gerrit 26610 0 master MERGED engine: RemoveNetworksByLabelParametersBuilder accepts any vlan device name Never
oVirt gerrit 26611 0 master MERGED engine: Adjust DetachNetworkFromVdsInterfaceCommand to accept any vlan name Never
oVirt gerrit 26612 0 master MERGED engine: Adjust GetAllChildVlanInterfacesQuery to accept any vlan name Never
oVirt gerrit 26613 0 master MERGED engine: Adjust GetAllSiblingVlanInterfacesQuery to accept any vlan name Never
oVirt gerrit 26614 0 master MERGED engine: Adjust LabelNicCommand to accept any vlan device name Never
oVirt gerrit 26615 0 master MERGED engine: Adjust RemoveBondCommand to accept any vlan device name Never
oVirt gerrit 26616 0 master MERGED engine: Adjust SetupNetworks to accept any vlan device name Never
oVirt gerrit 26617 0 master MERGED engine: Adjust UpdateNetworkToVdsInterfaceCommand to accept any vlan device name Never
oVirt gerrit 26618 0 master MERGED engine: Adjust VdsUpdateRunTimeInfo to accept any vlan device name Never
oVirt gerrit 27150 0 master MERGED engine: Adjust GetVlanParentQuery to accept any vlan device name Never
oVirt gerrit 27151 0 master MERGED engine: Adjust NetworkMonitoringHelper to accept any vlan device name Never
oVirt gerrit 27152 0 master MERGED engine: refactoring NetworkUtils.getVlanDeviceName Never
oVirt gerrit 27153 0 master MERGED engine: Adjust NetworkUtils to accept any vlan device name - phase 2 Never
oVirt gerrit 27154 0 master MERGED webadmin: Accept vlan devices identified by any name Never
oVirt gerrit 27496 0 master MERGED restapi: Arbitrary vlan device name Never

Description Amador Pahim 2013-08-22 13:32:03 UTC
Currently ovirt-engine can not handle vlans with names not in standard "dev.VLANID" (e.g. eth0.10-fcoe, em1.myvlan10, vlan20, ...).

If host has a vlan already created with an alternative name, host can not get activated and engine shows the error:

2013-08-22 10:21:03,315 ERROR [org.ovirt.engine.core.vdsbroker.vdsbroker.GetCapabilitiesVDSCommand] (DefaultQuartzScheduler_Worker-29) Failed in GetCapabilitiesVDS method, for vds: vdsm-devel; host: 192.168.122.158
2013-08-22 10:21:03,316 ERROR [org.ovirt.engine.core.vdsbroker.vdsbroker.GetCapabilitiesVDSCommand] (DefaultQuartzScheduler_Worker-29) Command GetCapabilitiesVDS execution failed. Exception: NumberFormatException: For input string: "myvlan40"

engine is trying to deduct vlan id from vlan name. After split the name, engine is hoping for "myvlan40" as vlan ID, which is clearly not true.

Commit "8b2bbe6 [2/3] Add "vlanid" to VLAN information" introduced in vdsm the ability to report the vlan id as an independent field, regardless the vlan name. Now engine must be fixed to get vlan id from this new field instead of deducting it from vlan name.

Steps to Reproduce:
- Change Host to maintenance mode.
- Add a vlan with alternative name to the Host:
 # ip link add dev eth1.myvlan10 link eth1 name eth1.myvlan10 type vlan id 10
- Activate the Host.

Comment 1 lpeer 2013-08-27 05:55:29 UTC
A fix for this bug should be on the VDSM side. Reporting of the VLAN tag instead of the vlan device name.

Comment 2 Dan Kenigsberg 2013-08-27 20:38:42 UTC
(In reply to lpeer from comment #1)
> A fix for this bug should be on the VDSM side. Reporting of the VLAN tag
> instead of the vlan device name.

Livnat, I believe that the vdsm side is already done upstream:
http://gerrit.ovirt.org/18180
http://gerrit.ovirt.org/18181
http://gerrit.ovirt.org/18182
http://gerrit.ovirt.org/18449

and that we are left with collecting this information in Engine. Amador, haven't you started an Engine-side patch for this?

Comment 3 Amador Pahim 2013-08-27 20:58:35 UTC
(In reply to Dan Kenigsberg from comment #2)
> (In reply to lpeer from comment #1)
> > A fix for this bug should be on the VDSM side. Reporting of the VLAN tag
> > instead of the vlan device name.
> 
> Livnat, I believe that the vdsm side is already done upstream:
> http://gerrit.ovirt.org/18180
> http://gerrit.ovirt.org/18181
> http://gerrit.ovirt.org/18182
> http://gerrit.ovirt.org/18449
> 

Yes, upstream vdsm is reporting vlanid like this:
...
        vlans = {'eth0.10': {'addr': '',
                             'cfg': {},
                             'iface': 'eth0',
                             'ipv6addrs': [],
                             'mtu': '1500',
                             'netmask': '',
                             'vlanid': 10},   <----
...
> and that we are left with collecting this information in Engine. Amador,
> haven't you started an Engine-side patch for this?

No, I have not.

Comment 6 errata-xmlrpc 2015-02-11 17:54:27 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-0158.html

Comment 7 Pavel Zhukov 2015-03-05 08:26:55 UTC
*** Bug 1198005 has been marked as a duplicate of this bug. ***


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