Re: [devel] [PATCH 0 of 1] Review Request for imm: Discard dying admin owners when headless [#1696]

2016-03-08 Thread Neelakanta Reddy
Hi hung, Reviewed the patch. Ack. /Neel. On Tuesday 08 March 2016 04:26 PM, Hung Nguyen wrote: > Summary: imm: Discard dying admin owners when headless [#1696] > Review request for Trac Ticket(s): 1696 > Peer Reviewer(s): Zoran, Neel > Pull request to: > Affected branch(es): 5.0 > Development br

Re: [devel] [PATCH 1 of 1] mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694]

2016-03-08 Thread A V Mahesh
Hi Vu, I was not intentional , I will update while pushing including with other occurrences. -AVM On 3/9/2016 7:58 AM, Vu Minh Nguyen wrote: > Hi Mahesh, > > I see you are/were using HOST_NAME_MAX in #1694/#1522. > > Is there any reason not using *_POSIX_HOST_NAME_MAX* as discussions in > atta

Re: [devel] [PATCH 1 of 1] mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694]

2016-03-08 Thread Vu Minh Nguyen
Hi Mahesh, I see you are/were using HOST_NAME_MAX in #1694/#1522. Is there any reason not using *_POSIX_HOST_NAME_MAX* as discussions in attached email? Regards, Vu. >-Original Message- >From: mahesh.va...@oracle.com [mailto:mahesh.va...@oracle.com] >Sent: Tuesday, March 08, 2016 10:46

Re: [devel] current opensaf-4.7.x (4.7.0-1) and default (5.0.M0-1 ) in-service upgrade is not working

2016-03-08 Thread A V Mahesh
Ok thanks , I over looked the ticket #1686 -AVM On 3/8/2016 11:37 AM, Vu Minh Nguyen wrote: > Hi Mahesh, > > I am able to reproduce the problem and found the root cause. > > I raised the ticket https://sourceforge.net/p/opensaf/tickets/1686 > > Regards, Vu. > > >> -Original Message- >

Re: [devel] [PATCH 0 of 1] Review Request for log: log agent library is not backward compatible [#1686]

2016-03-08 Thread Vu Minh Nguyen
Hi Lennart, Thanks for your comment. I will add the info when sending the patch for pushing. Regards, Vu. >-Original Message- >From: Lennart Lund [mailto:lennart.l...@ericsson.com] >Sent: Monday, March 07, 2016 7:18 PM >To: Vu Minh Nguyen; mathi.naic...@oracle.com >Cc: opensaf-devel@lis

Re: [devel] [PATCH 0 of 1] Review Request for imm: add check for SC absence in cold sync for IMMD [#1691]

2016-03-08 Thread Neelakanta Reddy
Hi Hung, Reviewed and tested the patch. Ack. /Neel. On Tuesday 08 March 2016 09:50 AM, Hung Nguyen wrote: > Summary: imm: add check for SC absence in cold sync for IMMD [#1691] > Review request for Trac Ticket(s): 1691 > Peer Reviewer(s): Neel > Pull request to: > Affected branch(es): 5.0 > Deve

Re: [devel] [PATCH 0 of 1] Review Request for imm: add check for 2PBE in cold sync for IMMD [#1695]

2016-03-08 Thread Neelakanta Reddy
Hi Zoran/Hung, Reviewed and tested the patch. Ack. /Neel. On Tuesday 08 March 2016 09:46 AM, Hung Nguyen wrote: > Summary: imm: add check for 2PBE in cold sync for IMMD [#1695] > Review request for Trac Ticket(s): 1695 > Peer Reviewer(s): Neel > Pull request to: > Affected branch(es): 4.6, 4.7,

Re: [devel] [PATCH 1 of 1] imm: add check for SC absence and 2PBE in cold sync for IMMD [#1625]

2016-03-08 Thread Hung Nguyen
Hi Neel, I will separate it into a new ticket for 2PBE. Thanks, Hung Nguyen - DEK Technologies From: Neelakanta Reddy reddy.neelaka...@oracle.com Sent: Monday, March 07, 2016 7:59PM To: Zoran Milinkovic zora

[devel] [PATCH 1 of 1] imm: add check for SC absence in cold sync for IMMD [#1691]

2016-03-08 Thread Hung Nguyen
osaf/services/saf/immsv/immd/immd_cb.h| 2 +- osaf/services/saf/immsv/immd/immd_evt.c | 3 --- osaf/services/saf/immsv/immd/immd_mbcsv.c | 29 - 3 files changed, 29 insertions(+), 5 deletions(-) SC absence allowed part will prevent that controllers don't ha

[devel] [PATCH 1 of 1] mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694]

2016-03-08 Thread mahesh . valla
osaf/libs/core/include/mds_papi.h | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) After the changes of MDS enhancement ticket #1522, MDS TCP application getting segmentation if application trying to send messge size > 65535 Bug : MDS:TCP code changes included the node name as a pa

[devel] [PATCH 1 of 1] imm: Remove coordinator role when SC absence happens [#1692]

2016-03-08 Thread Hung Nguyen
osaf/services/saf/immsv/immnd/immnd_evt.c | 37 - osaf/services/saf/immsv/immnd/immnd_proc.c | 34 +-- 2 files changed, 56 insertions(+), 15 deletions(-) Set 'mIsCoord' to false when headless to avoid coordinator from restarting. Also handle

[devel] Test mail. please ignore

2016-03-08 Thread Mathivanan Naickan Palanivelu
-- Transform Data into Opportunity. Accelerate data analysis in your applications with Intel Data Analytics Acceleration Library. Click to learn more. http://makebettercode.com/inteldaal-eval

[devel] [PATCH 1 of 1] imm: add check for 2PBE in cold sync for IMMD [#1695]

2016-03-08 Thread Hung Nguyen
osaf/services/saf/immsv/immd/immd_cb.h| 2 +- osaf/services/saf/immsv/immd/immd_mbcsv.c | 32 +++ 2 files changed, 33 insertions(+), 1 deletions(-) When 2PBE misconfiguration on controllers is detected, IMMD will exit in cold sync. diff --git a/osaf/services/

[devel] [PATCH 0 of 1] Review Request for imm: add check for 2PBE in cold sync for IMMD [#1695]

2016-03-08 Thread Hung Nguyen
Summary: imm: add check for 2PBE in cold sync for IMMD [#1695] Review request for Trac Ticket(s): 1695 Peer Reviewer(s): Neel Pull request to: Affected branch(es): 4.6, 4.7, 5.0 Development branch: 5.0 Impacted area Impact y/n ---

[devel] [PATCH 1 of 1] imm: Discard dying admin owners when headless [#1696]

2016-03-08 Thread Hung Nguyen
osaf/services/saf/immsv/immnd/ImmModel.cc | 12 ++-- osaf/services/saf/immsv/immnd/ImmModel.hh | 2 +- 2 files changed, 7 insertions(+), 7 deletions(-) Admin owners are marked as dying when being hard finalized during sync. Normally the dying admo will be discarded in finalizeSync().

Re: [devel] Proof Of Concept patch reusing SG FSM code for better handling of transient nodes during headless state(was Re: [PATCH 01 of 15] amfd: Add support for cloud resilience at common libs [#162

2016-03-08 Thread praveen malviya
Please apply this patch on top of l to 4. Thanks Praveen On 08-Mar-16 5:08 PM, praveen malviya wrote: > Hi, > > Attached is the PoC patch that re-uses existing SG FSM code, by resuming > the SG FSM state after first controller comes up. > Thereby, the patch avoids rebooting of the node in transit

Re: [devel] [PATCH 0 of 3] Review Request for rde: Support AMF configurations containing more than two OpenSAF 2N SUs [#79]

2016-03-08 Thread Neelakanta Reddy
Hi Anders, While applying rde patch the following errors are observed: patching file osaf/libs/agents/infrastructure/rda/Makefile.am patching file osaf/libs/agents/infrastructure/rda/rda.h patching file osaf/libs/agents/infrastructure/rda/rda_papi.c patching file osaf/libs/core/include/rda_papi.h

[devel] [PATCH 0 of 1] Review Request for imm: Discard dying admin owners when headless [#1696]

2016-03-08 Thread Hung Nguyen
Summary: imm: Discard dying admin owners when headless [#1696] Review request for Trac Ticket(s): 1696 Peer Reviewer(s): Zoran, Neel Pull request to: Affected branch(es): 5.0 Development branch: 5.0 Impacted area Impact y/n

[devel] Proof Of Concept patch reusing SG FSM code for better handling of transient nodes during headless state(was Re: [PATCH 01 of 15] amfd: Add support for cloud resilience at common libs [#1620])

2016-03-08 Thread praveen malviya
Hi, Attached is the PoC patch that re-uses existing SG FSM code, by resuming the SG FSM state after first controller comes up. Thereby, the patch avoids rebooting of the node in transition. More about the patch below: With this approach SG FSM was successfully recovered for QUIESCED and QUIE

Re: [devel] [PATCH 1 of 1] amf: Support AMF configurations containing more than two OpenSAF 2N SUs [#79]

2016-03-08 Thread praveen malviya
Hi Anders, When applied over cloud resilience patches (#1620) failures observed for amf patch: Hunk #6 succeeded at 673 (offset 20 lines). 1 out of 6 hunks FAILED -- saving rejects to file osaf/services/saf/amf/amfd/main.cc.rej .. 1 out of 4 hunks FAILED -- saving rejects to file osaf/ser

[devel] [PATCH 0 of 1] Review Request for log: log agent library is not backward compatible [#1686] V2

2016-03-08 Thread Vu Minh Nguyen
Summary: log: log agent library is not backward compatible [#1686] Review request for Trac Ticket(s): #1686 Peer Reviewer(s): Lennart, Anders W, Mathi Pull request to: Lennart Affected branch(es): Default Development branch: Default Impacted area Impact y/n -

[devel] [PATCH 0 of 1] Review Request for mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694]

2016-03-08 Thread mahesh . valla
Summary:mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694] Review request for Trac Ticket(s): #1694 Peer Reviewer(s): Mathi Pull request to: <> Affected branch(es): 5.0 default Development branch: default Impacted area Impact y/n --

[devel] [PATCH 0 of 1] Review Request for imm: add check for SC absence in cold sync for IMMD [#1691]

2016-03-08 Thread Hung Nguyen
Summary: imm: add check for SC absence in cold sync for IMMD [#1691] Review request for Trac Ticket(s): 1691 Peer Reviewer(s): Neel Pull request to: Affected branch(es): 5.0 Development branch: 5.0 Impacted area Impact y/n D

[devel] [PATCH 0 of 1] Review Request for imm: Remove coordinator role when SC absence happens [#1692]

2016-03-08 Thread Hung Nguyen
Summary: imm: Remove coordinator role when SC absence happens [#1692] Review request for Trac Ticket(s): 1692 Peer Reviewer(s): Zoran, Neel Pull request to: Affected branch(es): 5.0 Development branch: 5.0 Impacted area Impact y/n ---

Re: [devel] current opensaf-4.7.x (4.7.0-1) and default (5.0.M0-1 ) in-service upgrade is not working

2016-03-08 Thread Vu Minh Nguyen
Hi Mahesh, I am able to reproduce the problem and found the root cause. I raised the ticket https://sourceforge.net/p/opensaf/tickets/1686 Regards, Vu. >-Original Message- >From: A V Mahesh [mailto:mahesh.va...@oracle.com] >Sent: Tuesday, March 08, 2016 1:01 PM >To: Vu Minh Nguyen; ope

Re: [devel] current opensaf-4.7.x (4.7.0-1) and default (5.0.M0-1 ) in-service upgrade is not working

2016-03-08 Thread A V Mahesh
Hi Vu, Is current opensaf-4.7.x (4.7.0-1) and default (5.0.M0-1 ) in-service upgrade working for you ? -AVM On 3/3/2016 2:32 PM, A V Mahesh wrote: > Hi Vu, > > You just take > > (5.0.M0-1 ) : hg clone http://hg.code.sf.net/p/opensaf/staging & `hg > update default` > (4.7.0-1) : `hg clone http

[devel] [PATCH 1 of 1] log: log agent library is not backward compatible [#1686]

2016-03-08 Thread Vu Minh Nguyen
osaf/libs/agents/saf/lga/lga_mds.c | 42 - 1 files changed, 36 insertions(+), 6 deletions(-) The log agent on defaul branch (5.0) was not backward compatible with the old OpenSAF versions. This fix does an workaround to make it work. diff --git a/osaf/libs/

Re: [devel] [PATCH 0 of 1] Review Request for log: log agent library is not backward compatible [#1686]

2016-03-08 Thread Mathivanan Naickan Palanivelu
Hi Vu, I was curious too to get a description of the issue and fix! Please send an updated patch. Thanks, Mathi. > -Original Message- > From: Vu Minh Nguyen [mailto:vu.m.ngu...@dektech.com.au] > Sent: Tuesday, March 08, 2016 8:20 AM > To: Lennart Lund; Mathivanan Naickan Palanivelu > Cc

Re: [devel] [PATCH 0 of 1] Review Request for log: Extend information about origin of log record [#1480]

2016-03-08 Thread Vu Minh Nguyen
Hi Lennart, Please see my responses inline, with [Vu]. Regards, Vu. >-Original Message- >From: Lennart Lund [mailto:lennart.l...@ericsson.com] >Sent: Thursday, March 03, 2016 10:38 PM >To: Vu Minh Nguyen; Anders Widell; mathi.naic...@oracle.com >Cc: opensaf-devel@lists.sourceforge.net; B