[devel] OpenSAF 5.1.RC2 tagged, change control applies

2016-09-20 Thread Anders Widell
Hi all! We have now tagged and released OpenSAF 5.1.RC2 (release candidate 2). To reduce the risk of introducing new faults close to the release, all branches except the default branch are now under change control. This means that only approved bug fixes and documentation-only changes may be

[devel] Review request for Doc: Extensions PR document for 5.1

2016-09-20 Thread Anders Widell
Hi! Please find the OpenSAF Extensions PR document for 5.1 release for review. I have updated Section 4.1 to list all the services that support the extended SaNameT type in the OpenSAF 5.1 release. regards, Anders Widell OpenSAF_Extensions_PR.odt Description:

[devel] [PATCH 0 of 1] Review Request for amf: add support for cluster reboot [#2053]

2016-09-20 Thread Hans Nordeback
Summary: amf: add support for cluster reboot Review request for Trac Ticket(s): #2053 Peer Reviewer(s): Nagu, Praveen, Gary, AndersW Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n

Re: [devel] [PATCH 1 of 1] MDS: Log TIPC dropped messages [#1957]

2016-09-20 Thread Hans Nordebäck
Hi Mahesh, I think only logging is needed as proposed in the patch, as some services are already handling dropped messages. This logging will help in trouble shooting. Keeping TIPC_DEST_DROPPABLE to true will only make TIPC to silently drop messages, the original problem persists and needs

Re: [devel] [PATCH 1 of 1] amf: fix activeCompName in csiStateDescriptor used in CSI SET cbk [#2021]

2016-09-20 Thread minh chau
Hi Praveen, ack for code review. Thanks, Minh On 15/09/16 19:41, praveen.malv...@oracle.com wrote: > osaf/libs/agents/saf/amfa/ava_hdl.cc | 28 +++- > osaf/libs/common/amf/n2avamsg.c | 14 ++ > osaf/services/saf/amf/amfd/util.cc | 2 +- >

Re: [devel] [PATCH 1 of 1] MDS: Log TIPC dropped messages [#1957]

2016-09-20 Thread A V Mahesh
HI Anders Widell / HansN, On 9/16/2016 2:03 PM, Anders Widell wrote: > The idea was to just log reception of error info messages, for > trouble-shooting purposes. After multiple attempts, i manged to simulate TIPC_ERR_OVERLOAD error.After TIPC_ERR_OVERLOAD error is hit the cluster going

Re: [devel] [PATCH 1 of 1] clm: fill SaNameT value with zeros [#2015]

2016-09-20 Thread Mathivanan Naickan Palanivelu
Ack, Mathi. > -Original Message- > From: Zoran Milinkovic [mailto:zoran.milinko...@ericsson.com] > Sent: Tuesday, September 20, 2016 2:26 PM > To: Mathivanan Naickan Palanivelu > Cc: opensaf-devel@lists.sourceforge.net > Subject: [PATCH 1 of 1] clm: fill SaNameT value with zeros [#2015] >

[devel] [PATCH 1 of 1] clm: fill SaNameT value with zeros [#2015]

2016-09-20 Thread Zoran Milinkovic
osaf/libs/common/clmsv/clmsv_enc_dec.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) Set zeros in SaNameT value before SaNameT is populated with incoming SaNameT string from the network. This will make node name unique in patricia tree, and the node name search mismatch will be

[devel] [PATCH 0 of 1] Review Request for clm: fill SaNameT value with zeros [#2015]

2016-09-20 Thread Zoran Milinkovic
Summary: clm: fill SaNameT value with zeros [#2015] Review request for Trac Ticket(s): 2015 Peer Reviewer(s): Mathi Pull request to: Zoran Affected branch(es): opensaf-5.1.x, default(5.2) Development branch: default(5.2) Impacted area Impact y/n

Re: [devel] [PATCH 1 of 1] [mq]: 2032_fix_ckpttest.patch

2016-09-20 Thread A V Mahesh
Hi, ACK, Not tested , change the subject cpsvtest. -AVM On 9/20/2016 11:51 AM, Hoang Vo wrote: > tests/cpsv/test_cpa.c | 2 +- > tests/cpsv/test_cpa_util.c | 6 +++--- > 2 files changed, 4 insertions(+), 4 deletions(-) > > > diff --git a/tests/cpsv/test_cpa.c

Re: [devel] [PATCH 1 of 1] amfd: do not flush imm jobs on active controller[#2016]

2016-09-20 Thread minh chau
Hi Praveen, ack for code review. Thanks, Minh On 09/09/16 22:15, praveen.malv...@oracle.com wrote: > osaf/services/saf/amf/amfd/imm.cc | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > > AMF pushes IMM updates in a queue to be updated after finishing main job. > currently,

[devel] [PATCH 0 of 1] Review Request for ckpt: ckpttest for long dn 5_55 5_57 7_12 is failing #2032

2016-09-20 Thread Hoang Vo
Summary: ckpt: ckpttest for long dn is failing #2032 Review request for Trac Ticket(s): #2032 Peer Reviewer(s): mahesh.va...@oracle.com; anders.wid...@ericsson.com Pull request to: mahesh.va...@oracle.com Affected branch(es): default Development branch: default

[devel] [PATCH 1 of 1] [mq]: 2032_fix_ckpttest.patch

2016-09-20 Thread Hoang Vo
tests/cpsv/test_cpa.c | 2 +- tests/cpsv/test_cpa_util.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/cpsv/test_cpa.c b/tests/cpsv/test_cpa.c --- a/tests/cpsv/test_cpa.c +++ b/tests/cpsv/test_cpa.c @@ -45,7 +45,7 @@ const char *saf_error_string[] = {