Re: [Users] oVirt 3.1 - Error adding host , incorrect vdsm version

On Mon, Jun 18, 2012 at 04:08:06PM -0500, Trey Dockendorf wrote: > On Mon, Jun 18, 2012 at 3:25 PM, Dan Kenigsberg <danken(a)redhat.com&gt; > wrote: > > > On Mon, Jun 18, 2012 at 03:01:10PM -0500, Trey Dockendorf wrote: <snip> > > > > You are right. And it is truly surprising that this bug hasn't > > been > > open since we've bumped vdsm version to 4.10. > > > > Would you open up this bug, and make it block > > Bug 822145 - Tracker: oVirt 3.1 release > > ? > > > > The bootstrapping code is a mess. It evolves by adding one ugly > > patch > > over the other. In that spirit, would the following patch solve > > your > > bootstrapping pains? I actually really really hate the code there, and spent some time suggesting a less bad approach. Would you be kind enough to test http://gerrit.ovirt.org/#/c/5469/ and the 2 patches below it? > > > > diff --git a/vds_bootstrap/vds_bootstrap.py > > b/vds_bootstrap/vds_bootstrap.py > > index 0df5023..4fcc040 100755 > > --- a/vds_bootstrap/vds_bootstrap.py > > +++ b/vds_bootstrap/vds_bootstrap.py > > @@ -76,11 +76,11 @@ rhel6based = > > deployUtil.versionCompare(deployUtil.getOSVersion(), "6.0") >= 0 > > > > # TODO this is an infra-hackish heuristic for identifying Fedora > > # drop as soon as possible > > -fedorabased = > > deployUtil.versionCompare(deployUtil.getOSVersion(), "16") > > >= 0 > > +fedorabased = > > deployUtil.versionCompare(deployUtil.getOSVersion(), "17") > > >= 0 > > > > if rhel6based: > > VDSM_NAME = "vdsm" > > - VDSM_MIN_VER = VDSM_NAME + "-4.9" > > + VDSM_MIN_VER = VDSM_NAME + "-4.10" > > KERNEL_VER = "2.6.32-.*.el6" > > KERNEL_MIN_VER = 150 > > MINIMAL_SUPPORTED_PLATFORM = "6.0" > > > > > > Those changes worked, the install is now past that point and > installing > packages. > > Bug filed, https://bugzilla.redhat.com/show_bug.cgi?id=833201 Thanks! _______________________________________________ Users mailing list Users(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/users