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

[Dan Kenigsberg]

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 at redhat.com> 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!