[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--