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

2016-03-22 Thread Neelakanta Reddy
Hi Hung, Reviewed and tested the patch. Ack. /Neel. On Tuesday 08 March 2016 10:53 AM, Hung Nguyen wrote: > osaf/services/saf/immsv/immnd/immnd_evt.c | 37 > - > osaf/services/saf/immsv/immnd/immnd_proc.c | 34 +-- > 2 files changed, 56

Re: [devel] [PATCH 1 of 1] imm: Fix testcases that release admin owner from safImm object [#1708]

2016-03-22 Thread Zoran Milinkovic
Hi Hung, Very nice fix. Reviewed the patch. Ack from me. Thanks, Zoran -Original Message- From: Hung Nguyen [mailto:hung.d.ngu...@dektech.com.au] Sent: Tuesday, March 22, 2016 12:28 PM To: Zoran Milinkovic; reddy.neelaka...@oracle.com Cc: opensaf-devel@lists.sourceforge.net Subject: [

[devel] [PATCH 0 of 1] Review Request for imm: Fix testcases that release admin owner from safImm object [#1708]

2016-03-22 Thread Hung Nguyen
Summary: imm: Fix testcases that release admin owner from safImm object [#1708] Review request for Trac Ticket(s): 1708 Peer Reviewer(s): Zoran, Neel Pull request to: Affected branch(es): 4.7, 5.0 Development branch: 5.0 Impacted area Impact y/n

[devel] [PATCH 1 of 1] imm: Fix testcases that release admin owner from safImm object [#1708]

2016-03-22 Thread Hung Nguyen
tests/immsv/management/test_saImmOmClassCreate_2.c | 6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) Fix testcases that release admin owner from safImm object. diff --git a/tests/immsv/management/test_saImmOmClassCreate_2.c b/tests/immsv/management/test_saImmOmClassCreate_2.c --- a

Re: [devel] [PATCH 1 of 1] imm: Wait for veterans when IMMD starts [#1698]

2016-03-22 Thread Hung Nguyen
Hi Neel, I included your comment when pushing. Thanks, Hung Nguyen - DEK Technologies From: Neelakanta Reddy reddy.neelaka...@oracle.com Sent: Monday, March 21, 2016 6:09PM To: Hung Nguyen, Zoran Milinkovic h

Re: [devel] Review Request for log: update PR document for cloud resilience [#1687]

2016-03-22 Thread Vu Minh Nguyen
Hi Mathi, I am not sure if I understood your comments. With the latest update PR I sent to you, I did not mention about log file changes as "the information is too "low level" for the document, Lennart's comment. Regards, Vu. >-Original Message- >From: Mathivanan Naickan Palanivelu [m

Re: [devel] [PATCH 1 of 1] imm: Wait for veterans when IMMD starts [#1698]

2016-03-22 Thread Zoran Milinkovic
Hi Hung, Reviewed and tested the patch. Cannot reproduce the problem. Ack from me. Thanks, Zoran -Original Message- From: Hung Nguyen [mailto:hung.d.ngu...@dektech.com.au] Sent: Thursday, March 10, 2016 8:27 AM To: Zoran Milinkovic; reddy.neelaka...@oracle.com Cc: opensaf-devel@lists.

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

2016-03-22 Thread Zoran Milinkovic
Hi Hung, Reviewed the patch. Ac from me. Thanks, Zoran -Original Message- From: Hung Nguyen [mailto:hung.d.ngu...@dektech.com.au] Sent: Tuesday, March 08, 2016 11:57 AM To: Zoran Milinkovic; reddy.neelaka...@oracle.com Cc: opensaf-devel@lists.sourceforge.net Subject: [PATCH 1 of 1] imm:

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

2016-03-22 Thread Zoran Milinkovic
Hi Hung, Ack from me. Thanks, Zoran From: Hung Nguyen [mailto:hung.d.ngu...@dektech.com.au] Sent: Tuesday, March 22, 2016 4:46 AM To: Zoran Milinkovic; reddy.neelaka...@oracle.com Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [PATCH 1 of 1] imm: Remove coordinator role when SC absence ha

[devel] [PATCH 0 of 1] Review Request for core: Use std::atomic for counter no_of_msgs_sent [#1520]

2016-03-22 Thread Hans Nordeback
Summary: core: Use std::atomic for counter no_of_msgs_sent in unit test Review request for Trac Ticket(s): #1520 Peer Reviewer(s): Mathi, Ramesh Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n --

[devel] [PATCH 1 of 1] core: Use std::atomic for counter no_of_msgs_sent [#1520]

2016-03-22 Thread Hans Nordeback
osaf/libs/core/leap/tests/sysf_ipc_test.cc | 31 - 1 files changed, 9 insertions(+), 22 deletions(-) diff --git a/osaf/libs/core/leap/tests/sysf_ipc_test.cc b/osaf/libs/core/leap/tests/sysf_ipc_test.cc --- a/osaf/libs/core/leap/tests/sysf_ipc_test.cc +++ b/osaf/libs