[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Cluster-devel] joining the gitfence-agents group

thanks, i was doing some thinking the other night, given the goal is for this agent to work with oVirt, i'm not sure that all the oVirt required commands are supported in the agent code for this to work.
Also there's a comment about the port being required as a large path, this is very unlikely to work with oVirt, I need to consult that group to find out and perhaps submit a newer version.

On Fri, Mar 21, 2014 at 10:50 AM, Marek Grac <mgrac redhat com> wrote:

On 03/19/2014 10:26 PM, David Smith wrote:
Hi marek, sure, no problem, here's the changes; also sent to cluster-devel.

thanks, for next time it will be easier to you and other reviewers if you will sent a patch in a standard format. Take a look at

I've already tested by manually running the fence-raritan agent that this is working and functional

code looks pretty good and after few minor changes it can be accepted.

These are my comments:
    * please remove commented line of source code
    * exception checking 'Due to limitations ...' is not required any more because we have subpackage for every fence-agent so dependencies can be solved properly
    * default value can be now set more easier (--ipport in this case)
        all_opt["ipport"]["default"] = "23"
    * please add support also for delay before login which is very useful for 2-node clusters
        if options["--action"] in ["off", "reboot"]:


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]