<html><body> <tt>libvirt-cim-bounces@redhat.com wrote on 2008-04-24 04:30:39: > Guo Lian Yun wrote: > > libvirt-cim-bounces@redhat.com wrote on 2008-04-21 23:31:45: > > > > > Guo Lian Yun wrote: > > > > Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> > > > > > > EAFP 01_forward.py and EAFP 02_reverse.py both fail for me on F9 with > > > KVM and updated libvirt-cim sources. > > > > Could you please send the error log to me? Or do you know why these > > are failing? It pass for me on F9 with KVM and recent libvrit-cim sources. > > Here's the logs - I should have included them originally. > > EAFP 01 is failing due to the disk pool issue. You'll need to create a > function that makes sure the guest's disk resource belongs to a pool. > I'd suggested this in a previous mail, and I know you said you were > going to add such a function. But I haven't seen a patch for that yet - > did I miss it?</tt> <tt>I've created diskpool.conf file in EAFP.01 patch as follows. Any other </tt> <tt>file have to be created? </tt> <tt> </tt> <tt>+def conf_file(): + """ + Creating diskpool.conf file. + """ + try: + f = open(disk_file, 'w') + f.write('%s %s' % (test_dpath, '/')) + f.close() + except Exception,detail: + Globals.logger.error("Exception: %s", detail) + status = SKIP + sys.exit(status) + +def clean_up_restore(): + """ + Restoring back the original diskpool.conf + file. + """ + try: + if os.path.exists(back_disk_file): + os.remove(disk_file) + move_file(back_disk_file, disk_file) + except Exception, detail: + Globals.logger.error("Exception: %s", detail) + status = SKIP + sys.exit(status)</tt> <tt> > > Also, what version of libvirt are you using? The disk pool support is > new in 0.4.0. I'd assume that F9 would have libvirt 0.4.0 or higher. > It's possible your F8 system does not have a new version, which would > cause this test to pass. > > ElementAllocatedFromPool - 01_forward.py: FAIL > ERROR - ElementAllocatedFromPool returned 0 ResourcePool > objects for domain 'hd_domain' > CIM_ERR_FAILED: Unknown pool membership for `hd_domain/hda' > > ----- > > ElementAllocatedFromPool - 02_reverse.py: FAIL > ERROR - AttributeError : 'list' object has no attribute 'InstanceID' > CIM_ERR_NOT_FOUND: No such instance (foo) > > This is because the InstanceID used for the diskpool is incorrect. This > InstanceID will work on libvirt versions older than 0.4.0, but newer > versions need to use the diskpool ID.</tt> <tt> </tt> <tt> If the libvirt 0.4.0 or newer, the disk pool is supported. But why I can't </tt> <tt> get th diskpool ID by wbemein? There is none output of wbemein below(the libvirt</tt> <tt> version is 0.4.1 on my test machine):</tt> <tt> </tt><tt>wbemein <a href="http://root:password@localhost:5988/root/virt:KVM_DiskPool">http://root:password@localhost:5988/root/virt:KVM_DiskPool</a></tt><tt> > > > > > > > > > EAFP 01 and 02 are failing on Xen. I'd like to see test cases fixed > > > before we add KVM and XenFV support - otherwise, we can't test the KVM > > > and XenFV support properly. > > > > They are fail for Xen on my own machine(F8), but pass on other machine > > on F8, also pass on XenFV tested by Deepti. Of course, I will look > > into this issue. > > > > I'm guessing the XenFV system probably has an older version of libvirt. > > Sorry for all the confusion! It's tricky making tests work for multiple > guest types, as well as with differing package versions. > </tt> <tt> The patch for KVM & XenFV support are all passed on my test machine, which</tt> <tt> is set up by F9, the libvirt version is either 0.4.0 or 0.4.1. So I don't</tt> <tt> reproduce the failing. Maybe I'm still in the confusion of how to make this </tt> <tt> patch work correctly, would you please give more detail instruction or </tt> <tt> add another patch for it?</tt> <tt> Thanks!</tt> <tt> > -- > Kaitlin Rupert > IBM Linux Technology Center > kaitlin@linux.vnet.ibm.com > > _______________________________________________ > Libvirt-cim mailing list > Libvirt-cim@redhat.com > <a href="https://www.redhat.com/mailman/listinfo/libvirt-cim">https://www.redhat.com/mailman/listinfo/libvirt-cim</a> </tt></body></html>