Ack, reviewed only
Thanks,
Hans
On 04/04/2014 10:45 AM, nagendr...@oracle.com wrote:
osaf/services/saf/avsv/avd/avd_node.c | 9 +
osaf/services/saf/avsv/avd/avd_su.c | 8
osaf/services/saf/avsv/avnd/avnd_comp.c | 2 +-
3 files changed, 18 insertions(+), 1
Ack, reviewed only
Thanks,
Hans
On 04/03/2014 08:25 AM, praveen.malv...@oracle.com wrote:
osaf/services/saf/amf/amfd/sg_nored_fsm.cc | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
Problem: Due to fault in stable state in NoRed model, if sponsor goes
unassigned then AMF is not
Summary: IMMTOOLS: fix errors compiled with 32-bit compiler [#838]
Review request for Trac Ticket(s): 838
Peer Reviewer(s): Neelakanta
Pull request to: Zoran
Affected branch(es): opensaf-4.4.x, default(4.5)
Development branch: default(4.5)
Impacted area
Hi, use ULL, e.g 0x123ULL for the constant. Or use UINT64_C(0x123) from
stdint.h/BR HansN
-Original Message-
From: Zoran Milinkovic [mailto:zoran.milinko...@ericsson.com]
Sent: den 8 april 2014 10:46
To: reddy.neelaka...@oracle.com
Cc: opensaf-devel@lists.sourceforge.net
Subject:
I'll change to ULL when I push the code.
Thanks,
Zoran
-Original Message-
From: Hans Nordebäck
Sent: den 8 april 2014 11:00
To: Zoran Milinkovic; reddy.neelaka...@oracle.com
Cc: opensaf-devel@lists.sourceforge.net
Subject: RE: [devel] [PATCH 1 of 1] IMMTOOLS: fix errors compiled with
Hi AndersBj,
Reviewed and tested the patch.
Ack.
while testing in one of the scenario(when a delay is introduced in slave
saImmOiRtObjectCreate_2 and saImmOmAdminOwnerSet) ERR_NOT_EXIST is returned.
syslog:
-
Apr 8 14:41:35 Slot-3 osafimmpbed: IN saImmRepositoryInit:
I Tested your patch and it works because of the bug in the existing code
where the AVD subpart version is incremented but not the Max version(in
previous release).
Any way,
Ideal way of implementing a change in messaging part between x and y is
1. Increment the subpart version of both the
Hi zoran,
Reviewed and tested the patch.
Ack.
while pushing follow the suggestion of HansN.
/Neel.
On Tuesday 08 April 2014 02:15 PM, Zoran Milinkovic wrote:
Summary: IMMTOOLS: fix errors compiled with 32-bit compiler [#838]
Review request for Trac Ticket(s): 838
Peer Reviewer(s): Neelakanta
Hi zoran,
Reviewed the patch.
Ack.
/Neel.
On Thursday 03 April 2014 03:03 PM, Zoran Milinkovic wrote:
Summary: IMM: fix memory leak in IMM [#834]
Review request for Trac Ticket(s): 834
Peer Reviewer(s): Neelakanta
Pull request to: Zoran
Affected branch(es): opensaf-4.3.x, opensaf-4.4.x,
Ack for all the three patches in this series.
Tested by bringing up two controllers one with version 4.3 (latest
change set) and another with 4.4 (latest change set).
Thanks,
Praveen
On 07-Apr-14 11:05 AM, Hans Feldt wrote:
osaf/libs/common/avsv/avsv_d2nedu.c| 35
Ack from me.
Reviewed and tested.
Find minor comments below in the code started with [Zoran]
Best regards,
Zoran
-Original Message-
From: Anders Björnerstedt
Sent: den 4 april 2014 14:53
To: reddy.neelaka...@oracle.com; Zoran Milinkovic
Cc: opensaf-devel@lists.sourceforge.net
Subject:
When IMM handle is initialized with lower version than A.2.14, after ccbApply,
ccbAbort should return ERR_VERSION, but SA_AIS_OK is returned.
When CCB is not applied, ccbAbort works fine and returns ERR_VERSION.
The rest of the code is reviewed and tested, and it's ok.
NACK until the version is
12 matches
Mail list logo