This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/723/ |
openlmi-mof-register.py (Diff revision 1) | |||
---|---|---|---|
84 | devnull = open("/dev/null", "w") |
I think that this shouldn't be here.
openlmi-mof-register.py (Diff revision 1) | |||
---|---|---|---|
91 | devnull.close() |
I think that this shouldn't be here.
openlmi-mof-register.py (Diff revision 1) | |||
---|---|---|---|
155 | # fallback solution: most likely things will break |
||
156 | log_msg("WARNING: Could not find the Pegasus interop namespace, using root/interop") |
||
157 | ret = "root/interop" |
At this point, I think it would be a good idea to exit() with an error message (maybe a broken installation?). Unless Pegasus can have no "root/interop" or "root/PG_InterOp" and still work properly.
- Jan Synacek
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
Description
Diffs
|