Hi,

On Mon, Sep 13, 2010 at 03:00:43PM +0900, Yoshihiko SATO wrote:
> Hi,
> 
> For external/riloe, I noticed the following.
> 
> * syntax error
> ----------
> [r...@srv01 ~]# /usr/lib64/stonith/plugins/external/riloe getconfignames
>   File "/usr/lib64/stonith/plugins/external/riloe", line 389
>     for reply in result:
>                        ^
> SyntaxError: invalid syntax
> ----------
> 
> I want you to apply the following.
> 
> diff -r /usr/lib64/stonith/plugins/external/riloe 
> /usr/lib64/stonith/plugins/external/riloe.org
> 388c388
> <         fatal("no response from %s within %0.2f s"%(rihost,(t_end-t_begin)))
> ---
> >         fatal("no response from %s within %0.2f s"%(rihost,(t_end-t_begin))

Thanks for spotting this.

> 
> * ssl module
> ----------
> [r...@srv01 ~]# /usr/lib64/stonith/plugins/external/riloe
> Traceback (most recent call last):
>   File "/usr/lib64/stonith/plugins/external/riloe", line 40, in ?
>     import ssl
> ImportError: No module named ssl
> 
> [r...@srv01 ~]# python
> Python 2.4.3 (#1, Jun 11 2009, 14:09:37)
> [GCC 4.1.2 20080704 (Red Hat 4.1.2-44)] on linux2
> Type "help", "copyright", "credits" or "license" for more information.
> >>> import ssl
> Traceback (most recent call last):
>   File "<stdin>", line 1, in ?
> ImportError: No module named ssl
> ----------
> 
> I'm using RHEL5.5.
> The version of the python is 2.4.3 that is default.
> The ssl module doesn't exist normally in python 2.4.
> Because ImportError occurs, the riloe plug-in cannot be used.
> The downward compatibility to less than python 2.5 is
> necessary, what do you think?

The ssl module is imported to support the new http proxy
functionality. I moved the import statement over there now.  If
Jochen is listening, perhaps ssl could be made optional. Or we
should switch to another method and replace (iloproxyhost,
iloproxyport) with iloproxyurl.

Cheers,

Dejan

> Regards,
> Yoshihiko SATO
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to