Re: [Cluster-devel] cluster4 dlm dlm_stonith ??? should it really fence by turning node off?

2012-11-05 Thread David Teigland
On Sat, Nov 03, 2012 at 03:58:28PM +0100, Jacek Konieczny wrote: Hello, The dlm_stonith fencing helper is really convenient when Pacemaker is in use. Though, it doesn't quite work as I would expect ??? when fencing is needed it requests a node to be turned off instead of rebooting. And it

Re: [Cluster-devel] cluster4 dlm: startup notification for systemd

2012-11-05 Thread David Teigland
On Sat, Nov 03, 2012 at 04:27:54PM +0100, Jacek Konieczny wrote: Hello, The two patches: [PATCH 1/2] --foreground option added to dlm_controld [PATCH 2/2] Startup notification by sd_notify() add startup notification for the systemd service unit. This way startup of services

[Cluster-devel] [PATCH 3/3] fence agents: Transfer SNMP fence agents to long-opts + transfer script

2012-11-05 Thread Marek 'marx' Grac
--- fence/agents/apc_snmp/fence_apc_snmp.py | 24 fence/agents/cisco_mds/fence_cisco_mds.py |6 ++-- fence/agents/ibmblade/fence_ibmblade.py |4 +- fence/agents/ifmib/fence_ifmib.py |6 ++--

[Cluster-devel] [PATCH 1/3] fence agents: Transfer non-SNMP fence agents to long-opts

2012-11-05 Thread Marek 'marx' Grac
--- fence/agents/alom/fence_alom.py |8 ++-- fence/agents/apc/fence_apc.py | 70 fence/agents/bladecenter/fence_bladecenter.py | 30 +- fence/agents/cisco_ucs/fence_cisco_ucs.py | 18 +++---

[Cluster-devel] [PATCH] dlm_stonith_{off, reboot} aliases for fence helper

2012-11-05 Thread Jacek Konieczny
Not so pretty hack to allow rebooting instead of halting the node fenced by dlm_stonith. 'dlm_stonith_reboot' can be now used instead of 'dlm_stonith' in the dlm.conf file to request node reboot. 'dlm_stonith_off' alias is also provided to explicitly request power-off. Signed-off-by: Jacek

Re: [Cluster-devel] cluster4 dlm dlm_stonith ??? should it really fence by turning node off?

2012-11-05 Thread Jacek Konieczny
My workaround for the missing functionality (or wrong default): [PATCH] dlm_stonith_{off,reboot} aliases for fence helper Greets, Jacek

Re: [Cluster-devel] [PATCH] dlm_stonith_{off, reboot} aliases for fence helper

2012-11-05 Thread David Teigland
On Mon, Nov 05, 2012 at 07:05:22PM +0100, Jacek Konieczny wrote: - rv = stonith_api_kick_helper(nodeid, 300, 1); + rv = stonith_api_kick_helper(nodeid, 300, turn_off); I'd like it to be reboot, but seeing the arg as bool off I figured the opposite would be on ... if you're saying that

Re: [Cluster-devel] [PATCH] dlm_stonith_{off, reboot} aliases for fence helper

2012-11-05 Thread Jacek Konieczny
On Mon, Nov 05, 2012 at 02:30:33PM -0500, David Teigland wrote: On Mon, Nov 05, 2012 at 07:05:22PM +0100, Jacek Konieczny wrote: - rv = stonith_api_kick_helper(nodeid, 300, 1); + rv = stonith_api_kick_helper(nodeid, 300, turn_off); I'd like it to be reboot, but seeing the arg as bool

[Cluster-devel] [PATCH] GFS2: Don't call file_accessed() with a shared glock

2012-11-05 Thread Benjamin Marzinski
file_accessed() was being called by gfs2_mmap() with a shared glock. If it needed to update the atime, it was crashing because it dirtied the inode in gfs2_dirty_inode() without holding an exclusive lock. gfs2_dirty_inode() checked if the caller was already holding a glock, but it didn't make sure