[OCF]Updates Resource Agent API
Horms
ocf@lists.community.tummy.com
Fri, 1 Aug 2003 11:11:57 +0900
--bg08WKrSYDhXBjb5
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On Wed, Jul 30, 2003 at 04:14:41PM -0400, Lars Marowsky-Bree wrote:
> Hi all,
>=20
> after the good progress we have made on the RA API at OLS, I've checked
> in the new versions to CVS.
>=20
> You can see them at http://www.opencf.org/cgi-bin/viewcvs.cgi/specs/ra/
Excellent. Here are a few things that caught my eye.
* Section 3.2
"The subdirectories allow the installation of multiple RAs for the same
type, but from different vendors or package versions.
TODO: As the resource type is also embedded in the metadata returned
from the RA, should this sentence go away and instead have the RM
discover the installed resource types?"
I don't see any advantage to this. The proposed directory
structure seems both logical and conenient.
* Section 3.4
Proposed spelling change:
idem-potent -> idempotent
* Section 3.4.5
Should this section on reload mention the possibility
of being mapped to start/stop as per recover?
* Section 3.5.2
This section specifies that "OCF_" is reserved.
My reading is that this implies that "OCF_" is reserved
for global atributes which conflicts with the "OCF_RESKEY_"
syntax for meta data in the previous section.
Perhaps this should be made clearer. "OCF_" is reserved
for OCF uses. And perhaps we need a separate space for=20
globals, for exampe OCF_GLOBAL_ or OCF_G_
* Section 3.6.2
Proposed rewording:
These exit status codes are the ones documented in the LSB 1.1.0
specification, with additional explanations of how they shall be used
by RAs. In general, all non-zero status codes shall indicate failure
in accordance to the best current practices.
* Section 5.1
Proposed rewording:
We have... -> This API has...
* Section 5.2
I don't understand "vs"
"An example of a vs fully supported by the system."
* Sections 5 and 5.1 appear to be duplicated
* Contributors
Simon Horms -> Simon Horman
--=20
Horms
--bg08WKrSYDhXBjb5
Content-Type: application/pgp-signature
Content-Disposition: inline
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)
iD8DBQE/Kcxqdu+M6Iexz7URAjD+AJ4k6QrqL6ufjjspdsM68SIQHs/clQCgg7GK
qoNy0FuBmlv/0RHh/8BNmfc=
=7tK3
-----END PGP SIGNATURE-----
--bg08WKrSYDhXBjb5--