Re: [devel] [PATCH 1/1] amf: add support for container/contained [#70]

2018-08-27 Thread Gary Lee
Hi Alex No, I just ran kill 10 times to escalate restart to failover. Do you have a really small probation time in your demo config? Gary On 28/8/18 4:09 am, Alex Jones wrote: G'day Gary,     I can't reproduce this. Do you have a script or something that reproduces it? Alex On

[devel] [PATCH 0/1] Review Request for plmd: fix crash when saPlmReadinessTrack is called in error [#2919]

2018-08-27 Thread Alex Jones
Summary: plmd: fix crash when saPlmReadinessTrack is called in error [#2919] Review request for Ticket(s): 2919 Peer Reviewer(s): mathi, ravi Pull request to: Affected branch(es): develop Development branch: ticket-2919 Base revision: fb4890756ebd14fbe40906d37962b9261ed9a282 Personal repository:

[devel] [PATCH 1/1] plmd: fix crash when saPlmReadinessTrack is called in error [#2919]

2018-08-27 Thread Alex Jones
plmd crashes when saPlmReadinessTrack is called with entities pointer set, but smaller than what plmd would return. In this case plmd is returning ERR_NO_SPACE, which is correct, but it is setting numberOfEntities without setting the entities pointer. This causes the edu routines to crash. It is

Re: [devel] [PATCH 0/1] Review Request for plm: correct first arguement of API saPlmEntityGroupAdd() in apitest [#1983]

2018-08-27 Thread Alex Jones
Hi, This test is currently not enabled in test_saPlmEntityGroupCreate.c. Can you please enable it as part of this ticket? Alex On 08/20/2018 07:37 AM, Meenakshi TK wrote: __ NOTICE: This email was

Re: [devel] [PATCH 1/1] log: fix timeout when deleting configuration stream [#2903]

2018-08-27 Thread Lennart Lund
Hi Canh, Ack Thanks Lennart > -Original Message- > From: Canh Van Truong > Sent: den 24 augusti 2018 10:57 > To: Lennart Lund ; Vu Minh Nguyen > > Cc: opensaf-devel@lists.sourceforge.net; Canh Van Truong > > Subject: [PATCH 1/1] log: fix timeout when deleting configuration stream >

Re: [devel] [PATCH 0/2] Review Request for mdstest: correct timing issues in mdstest V3 [#2798]

2018-08-27 Thread Lennart Lund
Hi Hoa Le, Ack. See my comments. I think at least 1. has to be fixed but think about doing something about the other things as well No re-review is needed For detailed comments see the attached diff files [Lennart]. Comments summary: 1. Remove all out-commented code. Is still available in