Re: [devel] [PATCH 0 of 8] Review Request for CKPT: Support DNs longer than 255 bytes [#1574]

2016-06-30 Thread A V Mahesh
Hi Hoang, Thanks for the clarifications. Please hold on publishing new version of patch, I am in process of reviewing[PATCH 2 to 8] , so we may have some more comments , so we can have on single consolidated patch V2. -AVM On 7/1/2016 9:27 AM, Vo Minh Hoang wrote: > Dear Mahesh, > >

Re: [devel] Opensaf-devel Digest, Vol 37, Issue 62

2016-06-30 Thread Hans Nordebäck
Hi Alex, interesting, I was not aware of ticket #1431! I considered PLM but assumed it involves some more work than the patch I sent out, as this feature is to be delivered in August. The remote fencing patch could be replaced with your PLM solution when it is ready and PLM seems to be the

Re: [devel] Opensaf-devel Digest, Vol 37, Issue 62

2016-06-30 Thread Alex Jones
Yes, it uses libvirt for all the EE admin operations performed on the virtualized EE. There is no special fencing code in the PLM implementation, even for the current non-virtualized implementation. It's free with the PLM-EE hook in opensaf_reboot. So, if there are cases where opensaf_reboot

Re: [devel] Opensaf-devel Digest, Vol 37, Issue 62

2016-06-30 Thread Anders Widell
Ok, I see. Will PLM use libvirt for something else than fencing in such a configuration? regards, Anders Widell On 06/30/2016 05:02 PM, Alex Jones wrote: > Hi Anders, > > I have a basic working implementation which uses libvirt, but it's > not yet ready to send out for others to test. It

Re: [devel] Opensaf-devel Digest, Vol 37, Issue 62

2016-06-30 Thread Alex Jones
Hi Anders, I have a basic working implementation which uses libvirt, but it's not yet ready to send out for others to test. It relies on libvirt to talk to the hypervisor to tell it to fence the node. It uses the current setup in opensaf_reboot which calls PLM. Mathi and I have been

Re: [devel] Opensaf-devel Digest, Vol 37, Issue 62

2016-06-30 Thread Anders Widell
Interesting! How do you intend to implement this? Will OpenSAF rely on OpenHPI for remote fencing also when running inside virtual machines? Does OpenHPI support that today? regards, Anders Widell On 06/30/2016 04:11 PM, Alex Jones wrote: > Hi Hans, > > For what it is worth, there is a

Re: [devel] Opensaf-devel Digest, Vol 37, Issue 62

2016-06-30 Thread Alex Jones
Hi Hans, For what it is worth, there is a ticket (1431) which will implement virtualization of EEs in PLM, which is not currently supported. I am planning on implementing this at some point this summer. That should take care of fencing in a virtualized environment, without this patch, but

Re: [devel] [PATCH 0 of 1] Review Request for imm: Fix the startup delay in IMMD [#1896]

2016-06-30 Thread Neelakanta Reddy
Hi Hung, Reviewed the patch. Ack. /Neel. On 2016/06/28 09:19 AM, Hung Nguyen wrote: > Summary: imm: Fix the startup delay in IMMD [#1896] > Review request for Trac Ticket(s): 1896 > Peer Reviewer(s): Zoran, Neel > Pull request to: > Affected branch(es): 5.0, 5.1 > Development branch: 5.1 > >

Re: [devel] [PATCH 2 of 2] fm: Add support for remote fencing using STONITH [#1859]

2016-06-30 Thread Mathivanan Naickan Palanivelu
Okay, Thanks. I wil get back, Iam going through the documentation of the cluster-glue Iam wondering if a resource agent based approach might be more generic or extendable!? - Mathi. > -Original Message- > From: Hans Nordebäck [mailto:hans.nordeb...@ericsson.com] > Sent: Thursday, June

[devel] [PATCH 1 of 1] smfd:check smfNodeBundleActCmd at the time of modification

2016-06-30 Thread reddy . neelakanta
osaf/services/saf/smfsv/smfd/smfd_campaign_oi.cc | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) smfd will crash when smfNodeBundleActCmd is set to NULL, because of strdup. diff --git a/osaf/services/saf/smfsv/smfd/smfd_campaign_oi.cc

[devel] [PATCH 0 of 1] Review Request for smfd:check smfNodeBundleActCmd at the time of modification [#1903]

2016-06-30 Thread reddy . neelakanta
Summary:smfd:check smfNodeBundleActCmd at the time of modification [#1903] Review request for Trac Ticket(s): 1903 Peer Reviewer(s): Lennart, Rafael Affected branch(es): 4.7.x, 5.0.x, default Development branch: default Impacted area Impact y/n

Re: [devel] [PATCH 2 of 2] fm: Add support for remote fencing using STONITH V2 [#1859]

2016-06-30 Thread Hans Nordebäck
Hi Anders, please see my comments inlined. /Thanks HansN On 06/30/2016 10:31 AM, Anders Widell wrote: > Hi! > > Some initial thoughts on this: > > * In this patch the STONITH stuff is enabled by default, and the > PROMOTE_ACTIVE_TIMER is also enabled by default. I think we should > keep the

Re: [devel] [PATCH 2 of 2] fm: Add support for remote fencing using STONITH [#1859]

2016-06-30 Thread Hans Nordebäck
Hi Mathi, I run tests using xubuntu 14.04 with KVM and mainly used the man page for stonith. To install stonith on each virtual machine: sudo apt-get install cluster-glue I tested using both ssh and tcp. Tcp is easier to deploy, if a firewall is used add tcp port 16509 to the firewall rule

Re: [devel] [PATCH 1 of 2] fm: Add support for self-fencing [#1859]

2016-06-30 Thread Anders Widell
Hi! This patch is actually identical to the prototype code that I wrote and attached to the ticket, so I am not sure if I am supposed to also review it... anyways it is ack from from me for the first patch. :-) regards, Anders Widell On 06/23/2016 07:31 AM, Hans Nordeback wrote: >

Re: [devel] [PATCH 2 of 2] fm: Add support for remote fencing using STONITH V2 [#1859]

2016-06-30 Thread Anders Widell
Hi! Some initial thoughts on this: * In this patch the STONITH stuff is enabled by default, and the PROMOTE_ACTIVE_TIMER is also enabled by default. I think we should keep the default behaviour and not enable any of this by default. * It feels a bit odd that the opensaf_reboot script reads

Re: [devel] [PATCH 2 of 2] fm: Add support for remote fencing using STONITH [#1859]

2016-06-30 Thread Mathivanan Naickan Palanivelu
Hi Hans, Could you please give a pointer to the webpage of the stonith agents (and/or daemons?) that you used to test these changes? Thanks, Mathi. > -Original Message- > From: Hans Nordebäck [mailto:hans.nordeb...@ericsson.com] > Sent: Thursday, June 30, 2016 1:08 PM > To: Hans

Re: [devel] [PATCH 2 of 2] fm: Add support for remote fencing using STONITH [#1859]

2016-06-30 Thread Hans Nordebäck
Hi, anyone that had time to look at this patch? It would be good the get some early feedback as it may have to some further changes I considered if e.g. PLM should be used for the configuration but it seems to be more work, what do you say? /Thanks HansN -Original Message- From: Hans