Bug 1313881 - [networking] Adapt to the lack of CFG property of NICS in VDSM netinfo
Summary: [networking] Adapt to the lack of CFG property of NICS in VDSM netinfo
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-hosted-engine-setup
Classification: oVirt
Component: Network
Version: 2.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-4.1.0-alpha
: 2.1.0
Assignee: Simone Tiraboschi
QA Contact: Meni Yakove
URL:
Whiteboard:
: 1347438 (view as bug list)
Depends On:
Blocks: 1313872
TreeView+ depends on / blocked
 
Reported: 2016-03-02 14:22 UTC by Simone Tiraboschi
Modified: 2017-05-11 09:29 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-04-27 09:37:31 UTC
oVirt Team: Integration
Embargoed:
rule-engine: ovirt-4.1+
rule-engine: planning_ack+
sbonazzo: devel_ack+
mavital: testing_ack+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1347438 0 unspecified CLOSED vds_info.py: KeyError: 'cfg' 2021-02-22 00:41:40 UTC
oVirt gerrit 58931 0 master MERGED network: avoid using interface cfg entries 2016-09-12 16:27:13 UTC

Internal Links: 1347438

Description Simone Tiraboschi 2016-03-02 14:22:05 UTC 
Description of problem:
CFG property of NICS will be removed from VDSM netinfo. Adapt hosted-engine-setup code according to that.
Comment 1 Sandro Bonazzola 2016-03-09 14:49:51 UTC 
dan, can you please advise on what should be used instead of netinfo.CachingNetInfo ?
Comment 2 Dan Kenigsberg 2016-03-09 14:52:48 UTC 
We have seen that it is dangerous to depend on vdsm internal module structure. Vdsm has a public API, and only it should be used by components outside the Vdsm git tree. Vdsm has no test nor a commitment to keep internal functions, so please call getVdsCaps and parse it, just like Engine does.
Comment 3 Sandro Bonazzola 2016-05-02 10:04:43 UTC 
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.
Comment 4 Yaniv Lavi 2016-05-23 13:19:29 UTC 
oVirt 4.0 beta has been released, moving to RC milestone.
Comment 5 Yaniv Lavi 2016-05-23 13:23:13 UTC 
oVirt 4.0 beta has been released, moving to RC milestone.
Comment 6 Yaniv Lavi 2016-06-02 08:17:13 UTC 
What is the action item on this bug?
Comment 7 Simone Tiraboschi 2016-06-09 17:30:47 UTC 
In vdsm from branch 40, netinfo still contains the cfg element.

It has been recently removed on master: https://gerrit.ovirt.org/#/c/58226
but it sill requires some work to report otherwise missing information: https://gerrit.ovirt.org/#/c/58351/6
Comment 8 Douglas Schilling Landgraf 2016-06-17 16:41:13 UTC 
*** Bug 1347438 has been marked as a duplicate of this bug. ***
Comment 9 Yaniv Lavi 2016-09-08 08:30:51 UTC 
Can you add the ticket for adding the missing API info?
Comment 10 Dan Kenigsberg 2016-09-08 09:46:51 UTC 
Sorry, Yaniv, I do not understand your request. could you rephrase it?
Comment 11 Simone Tiraboschi 2016-09-08 09:50:01 UTC 
It was https://gerrit.ovirt.org/#/c/58351 but I see that now it's in.
Comment 12 Michael Burman 2017-04-04 11:05:13 UTC 
Hi Simone, 

Please explain how exactly this should be tested and verified? I see it has CodeChange Keywords. 

I don't see the cfg property on vdsCaps anymore, is it good enough?
If the change is on the hosted-engine code, then please move it to verified as i'm not sure what exactly should be tested here.
Comment 13 Dan Kenigsberg 2017-04-04 11:32:24 UTC 
QE needs to verify that all network-related hosted-engine flows still work.
There should be no functional change visible to the user.
Comment 14 Michael Burman 2017-04-04 14:31:19 UTC 
Tested with ovirt-hosted-engine-setup-2.1.0.5-1.el7ev.noarch
Note You need to log in before you can comment on or make changes to this bug.