Hi Canh,
I am not sure this is a correct fix. Please answer my comments [Lennart]:
Test case logtest 2 49 creates some cfg app to test the limitation, then
deleting these stream at the end. But the deleting fails due to timeout for
waiting the implementer. After that test case logtest 6 42 need
Ack, review only/Thanks HansN
-Original Message-
From: Gary Lee
Sent: den 1 augusti 2018 05:47
To: Hans Nordebäck ; Minh Hon Chau
Cc: opensaf-devel@lists.sourceforge.net; Gary Lee
Subject: [PATCH 1/1] amfd: remove redundant const_cast [#2907]
---
src/amf/amfd/clm.cc | 4 ++--
1
Summary: imm: syslog recent fevs evts when immnd down [#2898]
Review request for Ticket(s): 2898
Peer Reviewer(s): Hans Nordeback, Lennart Lund, Gary Lee, Vu Minh Nguyen
Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE ***
Affected branch(es): develop
Development branch: ticket-2898
Base
Hi Danh,
Ack with minor comments.
Regards, Vu
> -Original Message-
> From: Danh Vo
> Sent: Wednesday, August 1, 2018 3:29 PM
> To: hans.nordeb...@ericsson.com; lennart.l...@ericsson.com;
> gary@dektech.com.au; vu.m.ngu...@dektech.com.au
> Cc: opensaf-devel@lists.sourceforge.net;
Hi Guys,
I realized I forgot to put some notes in this review request...
75% of this code is from Praveen. I added some stuff that wasn't
there like shutting down contained sus when the container goes down,
allowing TRY_AGAIN in contained instantiation, and some more
Hi Lennart,
We have many classes which have empty value in tags.
If you look at the src/imm/config/SAI-AIS-IMM-XSD-A.02.17.xsd, there is no
restrictions on the length of
string value for tag. We should use minLength constraints
such as
`` if we do not allow empty value for
tag.
Since
Summary: imm: syslog recent fevs evts when immnd down [#2898]
Review request for Ticket(s): 2898
Peer Reviewer(s): Hans Nordeback, Lennart Lund, Gary Lee, Vu Minh Nguyen
Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE ***
Affected branch(es): develop
Development branch: ticket-2898
Base
An enhancement for IMM to print out 5 last FEVS events into syslog
when IMMND down for any reason.
---
src/imm/Makefile.am | 4 +-
src/imm/common/immsv_evt.c | 2 +-
src/imm/common/immsv_evt.h | 1 +
src/imm/immnd/immnd_evt.c| 6 ++
src/imm/immnd/immnd_utils.cc | 200
Hi Lennart
Please see my reply inline
Thanks
Canh
-Original Message-
From: Lennart Lund
Sent: Wednesday, August 1, 2018 9:20 PM
To: Canh Van Truong ; Vu Minh Nguyen
Cc: opensaf-devel@lists.sourceforge.net; Canh Van Truong
; Lennart Lund
Subject: RE: [PATCH 0/1] Review
Hi Vu
In what situation may a configuration xml file be created where a class
definition has a tag with no default value?
Does not seems correct and should probably not be allowed (loading such file
should fail). I there some reason for allowing that?
Thanks
Lennart
> -Original
10 matches
Mail list logo