Thanks for the cleanup

Seems to work flawlessly. Changed 4 additional lines (included in
minor_fix.patch) in order to make it work with the "stonith" program.
Else it would spew out:
Traceback (most recent call last):
 File "/usr/lib/stonith/plugins/external/riloe", line 44, in ?
   legacy_RI_HOST     = os.environ.get(RI_HOST,     None)
NameError: name 'RI_HOST' is not defined

Maybe someone with an older ILO version can test it too? Personally I
think it should work as well.


On 2/22/07, Tijl Van den Broeck <[EMAIL PROTECTED]> wrote:

I will test it asap and let you known.

--- lib/plugins/stonith/external/riloe.in	2007-02-22 11:07:37.000000000 +0100
+++ lib/plugins/stonith/external/riloe.in	2007-02-22 11:07:09.000000000 +0100
@@ -41,10 +41,10 @@
         print "Not enough arguments"
         sys.exit(1)
 
-legacy_RI_HOST     = os.environ.get(RI_HOST,     None)
-legacy_RI_HOSTRI   = os.environ.get(RI_HOSTRI,   None)
-legacy_RI_LOGIN    = os.environ.get(RI_HOST,     'Administrator')
-legacy_RI_PASSWORD = os.environ.get(RI_PASSWORD, '')
+legacy_RI_HOST     = os.environ.get('RI_HOST',     None)
+legacy_RI_HOSTRI   = os.environ.get('RI_HOSTRI',   None)
+legacy_RI_LOGIN    = os.environ.get('RI_LOGIN',     'Administrator')
+legacy_RI_PASSWORD = os.environ.get('RI_PASSWORD', '')
 
 reset_ok     = os.environ.get('ilo_can_reset',        '0')
 ilo_protocol = os.environ.get('ilo_protocol',         '1.2')
_______________________________________________________
Linux-HA-Dev: [email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to