This is an automatically generated e-mail. To reply, visit:

Ship it! (Diff revision 1)
    if (not just_mofs) and (version == None):
According to PEP8: "version == None" is wrong, use "version is None".

Looks good. Feel free to ship it now or add the database first.

- Radek Novacek

On August 15th, 2013, 3:50 p.m. CEST, Tomas Smetana wrote:

Review request for OpenLMI Developers.
By Tomas Smetana.

Updated Aug. 15, 2013, 3:50 p.m.

Repository: openlmi-providers


Here's a new registration script. It's a merge of the existing openlmi-mof-register and openlmi-register pegasus and should work as drop-in replacement for the openlmi-mof-register. The only slight difference is that the Pegasus interop namespace is not hard coded into the Pegasus registration calls but now there's an attempt to detect the right one.  And the whole thing is now written in (sort of) Python.

This is only the first step in the registration rewrite: Python should make it easier for manipulating a "side database" that would keep the registration data independently of the CIMOMs and eventually allow for re-registration when a new CIMOM is installed or upgraded.  This will also make possible for the providers package not to depend on a CIMOM.  I will prepare some design sketches of the registration process I envision to gather feedback and ideas: doing this correctly and robustly enough is going to be rather difficult.



View Diff