On Thu, Jun 21, 2012 at 5:31 AM, Dan Kenigsberg <danken(a)redhat.com&gt; wrote: > On Wed, Jun 20, 2012 at 05:58:57AM +0300, Dan Kenigsberg wrote: >> On Tue, Jun 19, 2012 at 02:43:14PM -0500, Trey Dockendorf wrote: >> > On Tue, Jun 19, 2012 at 1:40 PM, Dan Kenigsberg <danken(a)redhat.com&gt; wrote: >> > > On Tue, Jun 19, 2012 at 12:08:09PM -0400, Ofer Schreiber wrote: >> > >> Just checked this issue with latest ovirt-engine on F17 (with vdsm 4.10), and didn't encounter this issue at all. >> > >> >> > >> Danken - any idea why? >> > > >> > > Hey! I was the one soliciting ideas on how come this issue was not seen >> > > until Trey bumped on it. ;-) >> > > >> > > Maybe, just maybe, you host sees BOTH 4.10 and 4.9 ? >> > > >> > > what does the following print on your python interpreter? >> > > >> > > import yum >> > > my = yum.YumBase() >> > > my.pkgSack.searchNevra(name='rpm') >> > > >> > >> > >From the python , you want to output of name='rpm' or name='vdsm' ? I >> > figured the later but included both >> >> arghh s/rpm/vdsm/ >> >> Actually, this question was directed at Ofer. I was wodering how come >> the untouched code was working in his environment. > > For the record - we've tested this on Ofer's machine and found an old > yum repo with vdsm-4.9.3 or something, which hidden this bug from many > of us red-hatters. > > Thanks for reporting! The fix would be available in the next beta build. I've updated VDSM to 4.10.0-4 and now have run into a new, yet related issue. When the method yumSearchVersion is run, it fails with this error in the vds_installer log # cat vds_bootstrap.326741.log Fri, 06 Jul 2012 14:15:04 DEBUG **** Start VDS Validation **** Fri, 06 Jul 2012 14:15:04 DEBUG Entered VdsValidation(subject = '128.194.76.172', random_num = '8fe3c810-65bd-41e8-ae66-d4ea20fb71cb', rev_num = 'None', installVirtualizationService = 'True', installGlusterService = 'False') Fri, 06 Jul 2012 14:15:04 DEBUG Setting up Package Sacks Fri, 06 Jul 2012 14:15:04 DEBUG Host properly registered with RHN/Satellite. Fri, 06 Jul 2012 14:15:04 DEBUG <BSTRAP component='RHN_REGISTRATION' status='OK' message='Host properly registered with RHN/Satellite.'/> Fri, 06 Jul 2012 14:15:04 ERROR checkMajorVersion: Error searching for VDSM version! Traceback (most recent call last): File "/tmp/vds_bootstrap_8fe3c810-65bd-41e8-ae66-d4ea20fb71cb.py", line 250, in checkMajorVersion rc = deployUtil.yumSearchVersion(VDSM_NAME, VDSM_MIN_VER) File "/tmp/deployUtil.py", line 1049, in yumSearchVersion if rpmUtils.miscutils.compareVerOnly(pkg.ver, ver) >= 0: AttributeError: 'module' object has no attribute 'compareVerOnly' Fri, 06 Jul 2012 14:15:04 ERROR Unable to fetch VDSM with minimal version of 4.9. Please check if host is properly registered with updated yum repository Fri, 06 Jul 2012 14:15:04 DEBUG <BSTRAP component='VDSM_MAJOR_VER' status='FAIL' message='Unable to fetch VDSM with minimal version of 4.9. Please check if host is properly registered with updated yum repository'/> Fri, 06 Jul 2012 14:15:04 ERROR checkMajorVersion test failed Fri, 06 Jul 2012 14:15:04 DEBUG <BSTRAP component='RHEV_INSTALL' status='FAIL'/> Fri, 06 Jul 2012 14:15:04 DEBUG **** End VDS Validation **** Some research on rpmUtils.miscutils.compareVerOnly shows that it doesn't exist in yum-3.2.29 which is the latest in EL6 (CentOS 6.2). The only API documentation I found with that is yum-3.4.3. That being said, is this one of those cases where a patch is necessary to allow this code to work outside Fedora ? I believe a workaround exists in Dreyou's repo with a patch such as this

diff --git a/vdsm_reg/deployUtil.py.in b/vdsm_reg/deployUtil.py.in index 6bd4b3d..7dc049e 100644 --- a/vdsm_reg/deployUtil.py.in +++ b/vdsm_reg/deployUtil.py.in @@ -1046,7 +1046,8 @@ def yumSearchVersion(pkgName, ver): import rpmUtils.miscutils for pkg in yumListPackages(pkgName): - if rpmUtils.miscutils.compareVerOnly(pkg.ver, ver) >= 0: +# if rpmUtils.miscutils.compareVerOnly(pkg.ver, ver) >= 0: + if rpmUtils.miscutils.compareEVR(('', pkg.ver, ''), ('', ver, '')) >= 0: return True else: return False Thanks - Trey