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

See my comments marked [AndersW6]. regards, Anders Widell On 02/26/2016 10:31 AM, Nhat Pham wrote: > > Hi Mahesh, > > Please see my comment below with [NhatPham6] > > Best regards, > > Nhat Pham > > *From:*A V Mahesh [mailto:mahesh.va...@oracle.com] > *Sent:* Friday, February 26, 2016 12:31 PM >

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

Hi Mahesh, Please see my comment below with [NhatPham6] Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Friday, February 26, 2016 12:31 PM To: Nhat Pham ; 'Anders Widell' Cc:

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

Hi Nhat Pham, Please find my answers. -AVM On 2/26/2016 10:23 AM, Nhat Pham wrote: > > Hi Mahesh, > > Please see my answers below with [NhatPham5] > > Best regards, > > Nhat Pham > > *From:*A V Mahesh [mailto:mahesh.va...@oracle.com] > *Sent:* Friday, February 26, 2016 11:17 AM > *To:* Nhat

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

Hi Mahesh, Please see my answers below with [NhatPham5] Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Friday, February 26, 2016 11:17 AM To: Nhat Pham ; 'Anders Widell' Cc:

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

Hi Nhat Pham, >>[NhatPham4] To be more correct, the application will get >>SA_AIS_ERR_BAD_HANDLE when trying to access the lost checkpoint because all data was destroyed. >>[AndersW4] If this is a problem we could re-create the checkpoint with no sections in it. Even I come across this

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 comment below with [NhatPham4] Best regards, Nhat Pham From: Anders Widell [mailto:anders.wid...@ericsson.com] Sent: Thursday, February 25, 2016 9:25 PM To: A V Mahesh ; Nhat Pham Cc:

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

See my comments, marked [AndersW5]. regards, Anders Widell On 02/25/2016 11:09 AM, Nhat Pham wrote: > > Hi Mahesh and Anders, > > Please see my comment below with [NhatPham3] > > Best regards, > > Nhat Pham > > *From:*A V Mahesh [mailto:mahesh.va...@oracle.com] > *Sent:* Thursday, February 25,

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

Hi! See my comments inline, marked [AndersW4]. regards, Anders Widell On 02/25/2016 05:26 AM, A V Mahesh wrote: > Hi Nhat Pham, > > Please see my comment below. > > -AVM > > On 2/25/2016 7:54 AM, Nhat Pham wrote: >> >> Hi Mahesh, >> >> Would you agree with the comment below? >> >> To

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

Fine. Please publish the v3 patch. -AVM On 2/25/2016 3:50 PM, Nhat Pham wrote: > Hi Mahesh, > > Please see my answers below with [NhatPham4] > > Best regards, > Nhat Pham > > -Original Message- > From: A V Mahesh [mailto:mahesh.va...@oracle.com] > Sent: Thursday, February 25, 2016 4:31

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

Hi Mahesh, Please see my answers below with [NhatPham4] Best regards, Nhat Pham -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday, February 25, 2016 4:31 PM To: Nhat Pham ; 'Anders Widell' Cc:

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

Hi Mahesh and Anders, Please see my comment below with [NhatPham3] Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday, February 25, 2016 2:14 PM To: Nhat Pham ; 'Anders Widell' Cc:

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

>> With this patch the CPND detects un-recoverable checkpoints and deletes them all from the DB in case the headless state happens. By the way didn't tested some cases can you clarify below : which error will be revived by cpsv application of PL , for the unrecoverable checkpoint ?

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

Hi Nhat Pham, Please see my comment. -AVM On 2/25/2016 12:07 PM, Nhat Pham wrote: > > Hi Mahesh, > > Please see my comment below with [NhatPham2]. > > Best regards, > > Nhat Pham > > *From:*A V Mahesh [mailto:mahesh.va...@oracle.com] > *Sent:* Thursday, February 25, 2016 11:26 AM > *To:* Nhat

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

Hi Mahesh, Please see my comment below with [NhatPham2]. Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday, February 25, 2016 11:26 AM To: Nhat Pham ; 'Anders Widell' Cc:

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

Hi Nhat Pham, Please see my comment below. -AVM On 2/25/2016 7:54 AM, Nhat Pham wrote: > > Hi Mahesh, > > Would you agree with the comment below? > > To summarize, following are the comment so far: > > *Comment 1*: This functionality should be under checks if Hydra > configuration is enabled

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

Hi Mahesh, Would you agree with the comment below? To summarize, following are the comment so far: Comment 1: This functionality should be under checks if Hydra configuration is enabled in IMM attrName = const_cast("scAbsenceAllowed"). Action: The code will be updated

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

See my comments inline, marked [AndersW3]. regards, Anders Widell On 02/24/2016 07:32 AM, Nhat Pham wrote: > > Hi Mahesh and Anders, > > Please see my comments below. > > Best regards, > > Nhat Pham > > *From:*A V Mahesh [mailto:mahesh.va...@oracle.com] > *Sent:* Wednesday, February 24, 2016

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

Hi Mahesh and Anders, Please see my comments below. Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Wednesday, February 24, 2016 11:06 AM To: Nhat Pham ; 'Anders Widell' Cc:

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

Hi Nhat Pham, If component ( CPND ) restart allows while Controllers absent , before requesting CLM going to change return value to**SA_AIS_ERR_TRY_AGAIN , We need to get clarification from AMF guys on few things why because if CPND is on SA_AIS_ERR_TRY_AGAIN and component restart timeout

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

Hi Mahesh, Do you have any further comments? Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Monday, February 22, 2016 10:37 AM To: Nhat Pham ; 'Anders Widell' Cc:

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 1] Review Request for cpsv: Support preserving and recovering checkpoint replicas during headless state V2 [#1621]

Hi, >>According to the log, the PL-4 joined cluster, it means the cluster is not in headless state, doesn't it? Not exactly , best my knowledge CPSV application was running on PL4 ( cluster is up and running ) , the restarted both controllers it seem because of some other problem CPND

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

Hi Mahesh, Could you please clarify which case the error below happened? Feb 19 11:18:28 PL-4 osafimmnd[5422]: NO SERVER STATE: IMM_SERVER_SYNC_SERVER --> IMM_SERVER_READY Feb 19 11:18:28 PL-4 osafimmnd[5422]: NO Implementer connected: 45 (safClmService) <0, 2010f> Feb 19 11:18:28 PL-4

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

Hi, >>BTW, have you finished the review and test? I will finish by today. -AVM On 2/22/2016 7:48 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 1] Review Request for cpsv: Support preserving and recovering checkpoint replicas during headless state V2 [#1621]

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 [mailto:mahesh.va...@oracle.com] Sent: Friday, February 19, 2016 2:28 PM To: Nhat Pham ; 'Anders Widell'

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

Hi Nhat Pham, On 2/19/2016 12:28 PM, Nhat Pham wrote: > Could you please give more detailed information about steps to > reproduce the problem below? Thanks. Don't see this as specific bug , we need to see the issue as CLM integrated service point of view , by considering Anders Widell

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

Hi Mahesh, Could you please give more detailed information about steps to reproduce the problem below? Thanks. Best regards, Nhat Pham From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Friday, February 19, 2016 1:06 PM To: Anders Widell ; Nhat Pham

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

Hi Anders Widell, Thanks for the detailed explanation about CLM during headless state. HI Nhat Pham , Comment : 3 Please see below the problem I was interpreted now I seeing it during CLMD obscene ( during headless state ), so now CPND/CLMA need to to address below case , currently cpnd

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

Hi Nhat Pham, On 2/15/2016 3:08 PM, Nhat Pham wrote: > Hi Mahesh, > > It's good. Thank you. :) > > [AVM] Up on rejoining of the SC`s The replica should be re-created regardless > of another application opens it on PL4. >( Note : this comment is based on your explanation have not

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

Hi! Please find my answer inline, marked [AndersW]. regards, Anders Widell On 02/15/2016 10:38 AM, Nhat Pham wrote: > Hi Mahesh, > > It's good. Thank you. :) > > [AVM] Up on rejoining of the SC`s The replica should be re-created regardless > of another application opens it on PL4. >

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

Hi Mahesh, It's good. Thank you. :) [AVM] Up on rejoining of the SC`s The replica should be re-created regardless of another application opens it on PL4. ( Note : this comment is based on your explanation have not yet reviewed/tested , currently i am struggling

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

Hi Nhat Pham, How is your holiday went Please find my comments below On 2/15/2016 8:43 AM, Nhat Pham wrote: > Hi Mahesh, > > For the comment 1, the patch will be updated accordingly. [AVM] Please hold , I will provide more comments in this week , so we can have consolidated V3 > > For the

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

Hi Mahesh, For the comment 1, the patch will be updated accordingly. For the comment 2, I think the CKPT service will not be backward compatible if the scAbsenceAllowed is true. The client can't create non-collocated checkpoint on SCs. Furthermore, this solution only protects the CKPT service

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

Comment 2 : After incorporating the comment one all the Limitations should be prevented based on Hydra configuration is enabled in IMM status. Foe example : if some application is trying to create non-collocated checkpoint active replica getting generated/locating on SC then ,regardless of

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

To add: There are some get API, which don't depend upon the Director and information is at Node-Director, so in that case, what should be behaviour of those API in case of Headless. Can we have some README, which defines common guidelines for all SAF APIs. Or may be service specific also like

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

Hi! See my comments inline. regards, Anders Widell On 01/29/2016 10:35 AM, Nagendra Kumar wrote: > To add: There are some get API, which don't depend upon the Director and > information is at Node-Director, so in that case, what should be behaviour of > those API in case of Headless. If

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

Hi Anders, On 1/29/2016 1:02 PM, Nhat Pham wrote: > Hi Mahesh, > > As described in the README, the CKPT service returns SA_AIS_ERR_TRY_AGAIN > fault code in this case. > I guess it's same for other services. > > @Anders: Could you please confirm this? Please clarify on the SAF API`s behavior ,

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

Hi Mahesh, As described in the README, the CKPT service returns SA_AIS_ERR_TRY_AGAIN fault code in this case. I guess it's same for other services. @Anders: Could you please confirm this? Best regards, Nhat Pham -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com]

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

Hi, On 1/29/2016 11:45 AM, Nhat Pham wrote: >- The behavior of application will be consistent with other saf services > like imm/amf behavior during headless state. > [Nhat] I'm not clear what you mean about "consistent"? In the obscene of Director (SC's) , what is expected return values

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

Hi Nhat Pham, I stared reviewing this patch , so can please provide README file with scope and limitations , that will help to define testing/reviewing scope . Following are minimum things we can keep in mind while reviewing/accepting patch , - Not effecting existing functionality -

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

Hi Nhat Pham, Just came back from holidays, get back to you soon. -AVM On 1/14/2016 12:27 PM, Nhat Pham wrote: > Hi Mahesh, > > Would you have chance to review this patch? Thanks. > > Best regards, > Nhat Pham > > -Original Message- > From: Nhat Pham [mailto:nhat.p...@dektech.com.au] >

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

Hi Mahesh, Would you have chance to review this patch? Thanks. Best regards, Nhat Pham -Original Message- From: Nhat Pham [mailto:nhat.p...@dektech.com.au] Sent: Monday, January 4, 2016 4:45 PM To: mahesh.va...@oracle.com; anders.wid...@ericsson.com Cc: