Bug 995003 - 'Vm' object has no attribute 'addVmToMonitoredDomains'
Summary: 'Vm' object has no attribute 'addVmToMonitoredDomains'
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: vdsm
Version: 3.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.3.0
Assignee: Yeela Kaplan
QA Contact: Katarzyna Jachim
URL:
Whiteboard: storage
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-08 11:51 UTC by Katarzyna Jachim
Modified: 2016-02-10 17:11 UTC (History)
10 users (show)

Fixed In Version: is12
Doc Type: Bug Fix
Doc Text:
Trying to hot unplug a disk from a virtual machine failed with the error "'Vm' object has no attribute 'addVmToMonitoredDomains'". This is fixed by maintaining a list of the virtual machine's domains inside the virtual machine object, so hot unplugging a disk now works as expected.
Clone Of:
Environment:
Last Closed: 2014-01-21 16:30:42 UTC
oVirt Team: Storage
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
vdsm log (7.97 MB, text/x-log)
2013-08-08 11:51 UTC, Katarzyna Jachim
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2014:0040 0 normal SHIPPED_LIVE vdsm bug fix and enhancement update 2014-01-21 20:26:21 UTC

Description Katarzyna Jachim 2013-08-08 11:51:15 UTC
Created attachment 784321 [details]
vdsm log

Description of problem:
Hotplug of a disk fails with the following error:

Thread-520::ERROR::2013-08-08 10:54:18,006::BindingXMLRPC::993::vds::(wrapper) unexpected error
Traceback (most recent call last):
  File "/usr/share/vdsm/BindingXMLRPC.py", line 979, in wrapper
    res = f(*args, **kwargs)
  File "/usr/share/vdsm/BindingXMLRPC.py", line 268, in vmHotplugDisk
    return vm.hotplugDisk(params)
  File "/usr/share/vdsm/API.py", line 431, in hotplugDisk
    return curVm.hotplugDisk(params)
  File "/usr/share/vdsm/vm.py", line 3263, in hotplugDisk
    self.addVmToMonitoredDomains(self.id, diskParams['domainID'])
AttributeError: 'Vm' object has no attribute 'addVmToMonitoredDomains'

Version-Release number of selected component (if applicable): is9, vdsm-4.12.0-37.gitfd6a1b7.el6ev.x86_64

How reproducible: 100%
http://jenkins.qa.lab.tlv.redhat.com:8080/view/Storage/view/3.3/job/3.3-storage_hotplug_disk_hooks-iscsi-rest/45/

Steps to Reproduce:
1. try to hotplug a disk

Actual results: action fails

Expected results: action should succeed


Additional info:

Comment 1 Katarzyna Jachim 2013-08-08 13:21:53 UTC
[root@puma17 ~]# grep -r ToMonitoredDomains /usr/share/vdsm
/usr/share/vdsm/clientIF.py:    def addVmToMonitoredDomains(self, vmId, domains):
/usr/share/vdsm/vm.py:            self.cif.addVmToMonitoredDomains(self.id, domains)
/usr/share/vdsm/vm.py:                self.addVmToMonitoredDomains(self.id, diskParams['domainID'])

so probably it should be self.cif.addVmToMonitoredDomains(self.id, [diskParams['domainID']])

Comment 2 Allon Mureinik 2013-08-14 06:39:06 UTC
Seems to be a result of commit 87e61a0c57c1275f9cab3e523e02e51ed31c941f.
Yeela, please take a look?

Comment 3 Yeela Kaplan 2013-08-14 08:50:51 UTC
Actually, this was already fixed by 87e61a0c57c1275f9cab3e523e02e51ed31c941f.

Therefore, no longer relevant.

Comment 4 Allon Mureinik 2013-08-14 14:38:56 UTC
Moving to MODIFIED so QA have the chance to verify this when a build is released.

Comment 6 Charlie 2013-11-28 00:28:11 UTC
This bug is currently attached to errata RHBA-2013:15291. If this change is not to be documented in the text for this errata please either remove it from the errata, set the requires_doc_text flag to 
minus (-), or leave a "Doc Text" value of "--no tech note required" if you do not have permission to alter the flag.

Otherwise to aid in the development of relevant and accurate release documentation, please fill out the "Doc Text" field above with these four (4) pieces of information:

* Cause: What actions or circumstances cause this bug to present.
* Consequence: What happens when the bug presents.
* Fix: What was done to fix the bug.
* Result: What now happens when the actions or circumstances above occur. (NB: this is not the same as 'the bug doesn't present anymore')

Once filled out, please set the "Doc Type" field to the appropriate value for the type of change made and submit your edits to the bug.

For further details on the Cause, Consequence, Fix, Result format please refer to:

https://bugzilla.redhat.com/page.cgi?id=fields.html#cf_release_notes 

Thanks in advance.

Comment 7 errata-xmlrpc 2014-01-21 16:30:42 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.

http://rhn.redhat.com/errata/RHBA-2014-0040.html


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