All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek 'marx' Grác <mgrac@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] Eaton fence agent and more power devices support
Date: Thu, 20 Jan 2011 13:35:28 +0100	[thread overview]
Message-ID: <4D382C10.4000506@redhat.com> (raw)
In-Reply-To: <AANLkTimLwTz-poz35Y0i5GTnbrooo0sj=Drv7yczx9nu@mail.gmail.com>

Hi,

Arnaud Quette wrote:
> Hi Marek and the list,
>
> First of all, happy new year to you all: health, happiness and ... 
> high availability ;-)
>
> I've just derived a fence_eaton_snmp script, based on fence_apc_snmp.
> You'll find attached the diff against the latest fence-agents repository.
>
> This has been tested with the suitable hardware (Eaton Managed ePDU): 
> http://www.epdu.com
> Can you please merge this for the next cman releases.
>
Finally, I was able to review your code. I have few minor suggestions 
how to make code be better but we can solve that later. But I can do 
that later when we will have an access to this device. Fence agent will 
be part of the next fence-agents package.

> As a side note, I'm thinking about:
> - making one fence_pdu_snmp to group all this code (sysOID brings the 
> smartness of knowing which data should be used), or at least create 
> some common grounds to factorize code,

> - making a fence_nut_devices (or fence_power_devices) to use NUT [1] 
> to do the same for all supported serial / USB / SNMP / HTTP devices 
> (ie UPS, PDU and SCD).

I'm not against unfication but at least for power fencing - most of the 
shared code is already in fencing library and some agents differs a lot. 
Benefit of this simple agents is that we can't break anything else :) 
and code is already simple/small enough. In the case of SNMP based fence 
agents, it is possible that now as we have several such agents we can 
try to implement general solution.

m,



  reply	other threads:[~2011-01-20 12:35 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-11 15:53 [Cluster-devel] Eaton fence agent and more power devices support Arnaud Quette
2011-01-20 12:35 ` Marek 'marx' Grác [this message]
2011-01-24 12:18   ` Arnaud Quette
2011-01-24 12:47     ` Fabio M. Di Nitto
2011-01-24 13:19       ` Arnaud Quette
2011-01-24 13:31         ` Fabio M. Di Nitto

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D382C10.4000506@redhat.com \
    --to=mgrac@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.