Re: [devel] [PATCH 0 of 1] Review Request for imm: OiCcbObjectModifyCallback for appliers should include attributes that is not modified [#1651]

Hi Hung, Reviewed and tested the patch. Ack with the following comment. comment: 1. In getAllWritableAttributes function objName must be converted to nameToInternal. /Neel. On Friday 05 February 2016 03:57 PM, Hung Nguyen wrote: > Summary: imm: OiCcbObjectModifyCallback for appliers should

[devel] [PATCH 0 of 1] Review Request for log: fix info represents by millisecond token is not correct [#1619]

Summary: log: fix info represents by millisecond token is not correct [#1619] Review request for Trac Ticket(s): #1619 Peer Reviewer(s): Lennart, Anders W, Mathi, Vu Pull request to: Lennart Affected branch(es): 4.7, default Development branch: default Impacted

Re: [devel] [PATCH 0 of 5] Review Request for amf: Add support for cloud resilience [#1620] V2

Hi Nagu, Attached patch is for TC 41, 42. We have noticed one bug in sidep, will update it soon. Thanks, Minh On 22/02/16 23:48, Hans Nordebäck wrote: Hi, please see enclosed patch for TC #1, #6, #7, #8, #9, #10 and #11/Thanks HansN On 02/19/2016 10:09 AM, minh chau wrote: Hi Nagu,

[devel] [PATCH 0 of 1] Review Request for amf: Add support for cloud resilience [#1620] V3

Summary: amf: Add support for cloud resilience [#1620] V3 Review request for Trac Ticket(s): 1629 Peer Reviewer(s): Nagu, Praveen Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 1] amf: Add support for cloud resilience [#1620] V3

osaf/services/saf/amf/README-HEADLESS | 143 ++ 1 files changed, 143 insertions(+), 0 deletions(-) Update README-HEADLESS diff --git a/osaf/services/saf/amf/README-HEADLESS b/osaf/services/saf/amf/README-HEADLESS new file mode 100644 --- /dev/null +++

Re: [devel] [PATCH 0 of 5] Review Request for amf: Add support for cloud resilience [#1620] V2

Hi, please see enclosed patch for TC #1, #6, #7, #8, #9, #10 and #11/Thanks HansN On 02/19/2016 10:09 AM, minh chau wrote: Hi Nagu, Thanks for your testing. Below is our investigation from TC1 - TC31 which seem to be important, plus some patches that we're trying to fix the issues 1. IMM

Re: [devel] [PATCH 0 of 1] Review Request for cpsv: Support preserving and recovering checkpoint replicas during headless state V2 [#1621]

Hi! Please see my comments inline, marked [AndersW2]. regards, Anders Widell On 02/22/2016 03:18 AM, Nhat Pham wrote: > > Hi Mahesh and Anders, > > Please see my comment below. > > BTW, have you finished the review and test? > > Best regards, > > Nhat Pham > > *From:*A V Mahesh

Re: [devel] [PATCH 0 of 2] Review Request for clm: support simultaneous reboot of both controller nodes [#1646]

Ack, Let's revisit this patch later like for eg:- making it configurable, etc. Iam waiting to understand the big picture on how this feature has been coded in other services. Cheers, Mathi. - gary@dektech.com.au wrote: > HI Mathi > > Any comments regarding this series? Can we push