On Tue, Jun 22, 2010 at 04:43:28PM +0200, Guido Trotter wrote:
> On Wed, Jun 16, 2010 at 5:21 AM, Iustin Pop <[email protected]> wrote:
> > For this, we needed to extend the NodeImage class with a few extra
> > variables, and we do a trick in the node verification where we pick the
> > first node that returned valid OS data as the reference node, and then
> > we compare all other nodes against it.
> >
> > The checks added are:
> >
> > - consistency of DiagnoseOS responses
> > - multiple paths for an OS
> > - inconsistent OS between a reference node and the current node
> 
> LGTM (perhaps we should add the version check as well, if we make it
> in time for 2.2) (which would be good to avoid bumping up the api
> again soon).

We've documented it, so we can add it anytime - i.e. it is part of the
API 20, but we're not actually testing it, yet :)

iustin

Reply via email to