[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-ha-dev
Subject:    Re: [Linux-ha-dev] Fwd: [PATCH] call validate-all when
From:       Lars Marowsky-Bree <lmb () suse ! de>
Date:       2009-11-11 12:09:49
Message-ID: 20091111120949.GE14594 () suse ! de
[Download RAW message or body]

On 2009-11-11T12:22:48, Dejan Muhamedagic <dejanmm@fastmail.fm> wrote:

> > Simply put: if you're checking for dependencies possibly provided by
> > other resources, these won't be present at probe time. (Kind of obvious,
> > really.)
> 
> In this case, the CRM, which knows that dependencies are not
> started, should ignore the INSTALLED error.

No, that is silly, sorry. The RA simply shouldn't return the wrong
result.

For example, the ERR_INSTALLED error is quite proper to return if the
node doesn't ever have enough memory to support the resource, if it's
running the wrong kernel, the wrong architecture, a system binary is
missing or the wrong version ... In that case, one wouldn't want the CRM
to try and start there.

The distinction is quite similar to ERR_INSTALLED versus ERR_CONFIGURED;
the RA "simply" must return the proper value. It's almost like saying
the CRM should know that "ERR_GENERIC" on probe means stopped ...

> But that's probably an extra burden on an already complex piece of
> software. Or we provide some facility to all resource agents which
> would deal with the binaries on a missing filesystem situation. I
> mean, this is a issue common to all resource agents.

We already have check_binary and have_binary; adding "probe_binary" and
"probe_path" (for non-executables) would be trivial.

Still, the RAs would need to be fixed accordingly.


Regards,
    Lars

-- 
Architect Storage/HA, OPS Engineering, Novell, Inc.
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)
"Experience is the name everyone gives to their mistakes." -- Oscar Wilde

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

[prev in list] [next in list] [prev in thread] [next in thread]