Re: [devel] [PATCH 1/1] amfnd: handle component failover during SURestart [#3207]

2020-08-12 Thread Thang Duc Nguyen
Hi Minh, It can be reproduced. Refer on ticket for more info. See my respond inline. -Original Message- From: Minh Hon Chau Sent: Wednesday, August 12, 2020 4:24 PM To: Thang Duc Nguyen ; Thuan Tran Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [PATCH 1/1] amfnd: handle

Re: [devel] [PATCH 1/1] amfnd: handle component failover during SURestart [#3207]

2020-08-12 Thread minhchau
Hi Thang ack from me Thanks Minh On 12/8/20 7:58 pm, Thang Duc Nguyen wrote: Hi Minh, It can be reproduced. Refer on ticket for more info. See my respond inline. -Original Message- From: Minh Hon Chau Sent: Wednesday, August 12, 2020 4:24 PM To: Thang Duc Nguyen ; Thuan Tran Cc:

Re: [devel] [PATCH 1/1] amfnd: handle component failover during SURestart [#3207]

2020-08-12 Thread minhchau
Hi Thang, Is this reproducible? And one comment inline. Thanks Minh On 7/8/20 12:03 pm, thang.d.nguyen wrote: During SURestart, another compoenent is failovered. The SU-SI modify event message need buffering to process later. The new active assignment on SU need processing after it is

Re: [devel] [PATCH 1/1] rde: allow node start first to be promoted in relax mode [#3191]

2020-08-12 Thread Thang Duc Nguyen
Hi Thuan, I think the pending time only need updating/calculating in SendPeerInfoResp(), when it receives the peer up event. No need to update again in SetPeerState(). B.R/Thang -Original Message- From: Thuan Tran Sent: Monday, May 25, 2020 11:27 AM To: Minh Hon Chau ; Thang Duc

Re: [devel] [PATCH 1/1] rde: allow node start first to be promoted in relax mode [#3191]

2020-08-12 Thread Thuan Tran
Hi Thang, It is for the case as following. For example: Peer B see peer A up, calculate B pending time and SendPeerInfoResp() to peer A. If peer A get peer B info before see peer B up, A pending time is still zero now. Then A need calculate time (in SetPeerState if pending time is zero) to

Re: [devel] [PATCH 1/1] rde: allow node start first to be promoted in relax mode [#3191]

2020-08-12 Thread Thuan Tran
Hi Thang, See my reply inline. Best Regards, ThuanTr From: Thang Duc Nguyen Sent: Thursday, August 13, 2020 8:07 AM To: Thuan Tran ; Minh Hon Chau Cc: opensaf-devel@lists.sourceforge.net Subject: RE: [PATCH 1/1] rde: allow node start first to be promoted in relax mode [#3191] Hi Thuan, See

[devel] [PATCH 1/1] pyosaf: Support python3 [#3210]

2020-08-12 Thread thien.m.huynh
Adapt pyosaf to support python3, and be compatible with python2 --- python/pyosaf/saAis.py| 51 +++ python/pyosaf/saAmf.py| 8 +++-- python/pyosaf/saImm.py| 4 ++- python/pyosaf/saLog.py| 4 ++-

[devel] [PATCH 0/1] Review Request for pyosaf: Support python3 [#3210]

2020-08-12 Thread thien.m.huynh
Summary: pyosaf: Support python3 [#3210] Review request for Ticket(s): 3210 Peer Reviewer(s): Thuan, Thang, Minh Pull request to: Thuan Affected branch(es): develop Development branch: ticket-3210 Base revision: db5e0c24ff713126d2912b6c3ec59cfbcfbda491 Personal repository:

Re: [devel] [PATCH 1/1] rde: allow node start first to be promoted in relax mode [#3191]

2020-08-12 Thread Thang Duc Nguyen
Hi Thuan, See my comment inline. From: Thuan Tran Sent: Wednesday, August 12, 2020 10:25 PM To: Thang Duc Nguyen ; Minh Hon Chau Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [PATCH 1/1] rde: allow node start first to be promoted in relax mode [#3191] Hi Thang, It is for the case as