Hi Praveen,
Thank you for your comments.
New patch has been sent out for review. Document update will be sent for
review soon.
Best Regards,
Nguyen
On 1/31/2017 1:41 PM, praveen malviya wrote:
> Hi,
>
> Patch looks fine but some minor corrections are needed and some
> documentation is also re
The $piddir variable (containing path to amf_demo comp's pid file)
is missed to be defined in amf_demo_script.
This could lead to the amf_demo process not getting truely killed
in some cases when cleanup is called (e.g when invoking
saAmfComponentErrorReport()), leaving the process unmanaged by AM
n
Samples y
Tests n
Other n
Comments (indicate scope for each "y" above):
-
revision 842d06c3a6787789b018221ed5ee43ee02f48e0d
Author: Nguyen Luu
Date: Wed, 12 Apr 2017 14:2
Core libraries n
Samples n
Tests n
Other n
Comments (indicate scope for each "y" above):
-
revision 92f39d85213eea901334dcdf3c6b3ac7669810d1
Author: Nguyen Luu
Date: Thu, 13 Apr
Valid environment variable should have the format 'var=value'.
AMF currently does not validate this format during CREATE CCBs
for comptype and comp objects (MODIFY allowed for comp after #2255)
related to saAmfxxxCmdEnv attribute.
Besides, the existing validation in avnd_comp_clc_cmd_execute()
is
Valid environment variable should have the format 'var=value'.
AMF currently does not validate this format during CREATE CCBs
for comptype and comp objects (MODIFY allowed for comp after #2255)
related to saAmfxxxCmdEnv attribute.
Besides, the existing validation in avnd_comp_clc_cmd_execute()
is
-
revision 94ced33d5ee94030122a5431a3a131ed705736a6
Author: Nguyen Luu
Date: Fri, 28 Apr 2017 17:03:05 +0700
amf: validate env variable format set in comptype/comp objects [#2409]
Valid environment variable should have the format 'var=value'.
AMF currently does not validate this format during CREATE CCBs
for compt
Suite 35 of ntftest requires long-dns support be enabled in the system.
If ntftest is run without any argument or specific test suite, suite 35
should be skipped if long-dns support is not enabled in the system.
---
src/ntf/apitest/test_ntf_imcn.c | 72 ++---
1
-
revision 9d65e9adefe0b2651423b7cdf40f8dae1b13f69f
Author: Nguyen Luu
Date: Fri, 19 May 2017 15:13:40 +0700
ntf: ntftest to check longDnsAllowed to run or skip suite 35 [#2463]
Suite 35 of ntftest requires long-dns support be enabled in the system.
If ntftest is run without any argument or specific test suite, suite 35
Suite 35 of ntftest requires long-dns support be enabled in the system.
If ntftest is run without any argument or specific test suite, suite 35
should be skipped if long-dns support is not enabled in the system.
---
src/ntf/apitest/test_ntf_imcn.c | 70 ++---
1
-
revision 46f41bc31bc2c25346916a816144ec7370da7373
Author: Nguyen Luu
Date: Fri, 19 May 2017 15:47:20 +0700
ntf: ntftest to check longDnsAllowed to run or skip suite 35 [#2463]
Suite 35 of ntftest requires long-dns support be enabled in the system.
If ntftest is run without any argument or specific test suite, suite 35
When called with an uninitialized or already finalized handle,
saAmfPmStart_3 and saAmfResponse_4 should return SA_AIS_ERR_BAD_HANDLE
instead of SA_AIS_ERR_VERSION as previously done.
---
src/amf/agent/amf_agent.cc | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/amf/agent/a
-
revision 5c481c9e0fb45afe6cb9c7ae838191ff68b64b48
Author: Nguyen Luu
Date: Wed, 2 Aug 2017 10:11:02 +0700
amfa: Fix saAmfPmStart_3 and saAmfResponse_4 to correctly return BAD_HANDLE
[#2539]
When called with an uninitialized or already finalized handle,
saAmfPmStart_3 and saAmfResponse_4 should return SA_AIS_ERR_
-
revision a8676a873a86f9836e2a1a55e27c8f1acee0535f
Author: Nguyen Luu
Date: Wed, 2 Aug 2017 13:48:13 +0700
amfa: Fix saAmfComponentErrorClear_4 to return ERR_NOT_EXIST for non-exist comp
[#2540]
When called with a non-existing component name, saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST
When called with a non-existing component name, saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST instead of SA_AIS_ERR_BAD_OPERATION
as previously done.
---
src/amf/amfnd/err.cc | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/amf/amfnd/err.cc b/src/amf/am
And PRIx64 from instead of %llx?
/Regards HansN
On 08/02/2017 06:01 AM, Nguyen Luu wrote:
When called with an uninitialized or already finalized handle,
saAmfPmStart_3 and saAmfResponse_4 should return SA_AIS_ERR_BAD_HANDLE
instead of SA_AIS_ERR_VERSION as previously done.
---
src/amf/
7da04c1a07
Author: Nguyen Luu
Date: Fri, 11 Aug 2017 14:51:31 +0700
amfa: Fix api internal check to avoid fatal mutex unlock [#2548]
Current check for the required setting of the SA_AMF_COMPONENT_NAME
env variable in some amf api's (ComponentRegister, QuiescingComplete)
would crash the invo
Current check for the required setting of the SA_AMF_COMPONENT_NAME
env variable in some amf api's (ComponentRegister, QuiescingComplete)
would crash the invoking process if that env variable was missed
to be set for some reason, as the agent lib tries, during cleanup,
to unlock a mutex which it ha
-
revision 2df1d81a8b76eebf0f7fe8df25bc942e00c4aa7f
Author: Nguyen Luu
Date: Fri, 11 Aug 2017 15:23:24 +0700
amfa: Fix saAmfComponentErrorClear_4 to return ERR_NOT_EXIST for non-exist comp
[#2540]
When called with a non-existing component name, saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST
When called with a non-existing component name, saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST instead of SA_AIS_ERR_BAD_OPERATION
as previously done.
---
src/amf/amfnd/err.cc | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/amf/amfnd/err.cc b/src/amf/
Hi,
I'd like to push this patch tomorrow if there are no more comments.
Thanks,
Nguyen
On 8/7/2017 2:00 PM, Nguyen Luu wrote:
Thanks, Hans and Anders, for your review.
About the code refactoring, should we do it in a separate ticket for
all AMF API's that use such constant
When called with a non-existing component name, saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST instead of SA_AIS_ERR_BAD_OPERATION
as previously done.
---
src/amf/amfnd/err.cc | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/src/amf/amfnd/err.cc
-
revision 7d907166c175fef1ca5e1ddbb258a8b5d915a364
Author: Nguyen Luu
Date: Wed, 16 Aug 2017 14:48:56 +0700
amfa: Fix saAmfComponentErrorClear_4 to return ERR_NOT_EXIST for non-exist comp
[#2540]
When called with a non-existing component name, saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST
12:51 PM, Nguyen Luu wrote:
When called with a non-existing component name,
saAmfComponentErrorClear_4
should return SA_AIS_ERR_NOT_EXIST instead of SA_AIS_ERR_BAD_OPERATION
as previously done.
---
src/amf/amfnd/err.cc | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a
Hi,
Tested OK with legacy test. This change is similar to that in #2539
which was already pushed.
Thanks,
Nguyen
On 7/28/2017 4:41 PM, Nagendra Kumar wrote:
---
src/amf/agent/amf_agent.cc | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/amf/agent/amf_agent.cc b/src
-
revision 300b97ff3e52f29f9696a8b7e799af0880eb3b58
Author: Nguyen Luu
Date: Fri, 6 Oct 2017 13:49:30 +0700
pyosaf: Fix various pylint and PEP8 issues of pyosaf utils [#2603]
Complete diffstat:
--
python/pyosaf/utils/__init__.py | 48 +-
python/pyosaf/utils/clm/__init__.py
Hi,
My comments:
Why not just simply reusing the decorate() function as below, instead of
inventing a new initialize_decorate() function only for zzzInitialize()
API? And why making 'version' a global variable when it is only used by
initialize()?
/
//@decorate//
//def initialize(callbacks=No
-
revision 78efa1ad3c85e245a3e262c84662a3f0354f4436
Author: Nguyen Luu
Date: Wed, 18 Oct 2017 11:56:06 +0700
pyosaf: Fix various pylint and PEP8 issues of pyosaf utils [#2603]
Complete diffstat:
--
python/pyosaf/utils/__init__.py | 84 +--
python/pyosaf/utils/clm/__init__.py
uot;y" above):
-
revision 02c0cbb22ae82801ada8e206d09427516a0a848b
Author: Nguyen Luu
Date: Thu, 19 Oct 2017 13:23:57 +0700
pyosaf: Fix various pylint and PEP8 issues of pyosaf utils [#2603]
- Fix most pylint and PEP8 warnings of files under pyosaf/u
---
python/pylintrc | 380
1 file changed, 380 insertions(+)
create mode 100644 python/pylintrc
diff --git a/python/pylintrc b/python/pylintrc
new file mode 100644
index 000..c1021a1
--- /dev/null
+++ b/python/pylintrc
@@ -0,0 +1,380 @
uot; above):
-
revision 51ec9f79721a03210f2a693eb37cd2e0698fd2a5
Author: Nguyen Luu
Date: Thu, 19 Oct 2017 14:46:58 +0700
pyosaf: Create a common pylint config file for pyosaf [#2636]
Added Files:
python/pylintrc
Complet
Hi Hans,
I have one comment inline marked with [Nguyen].
Thanks,
Nguyen
On 10/23/2017 7:40 PM, Hans Nordeback wrote:
---
Makefile.am | 9 +-
python/pylintrc | 381
2 files changed, 389 insertions(+), 1 deletion(-)
create mode
-
revision 13615cadcdb4f433f00cb9504878312db68c41ad
Author: Nguyen Luu
Date: Wed, 25 Oct 2017 17:48:42 +0700
pyosaf: High level python interfaces for NTF [#2602]
Improved implementation of NTF pyosaf utils
The following comments shall be removed when the patch is pushed.
- This is the 2nd patch in the
Improved implementation of NTF pyosaf utils
The following comments shall be removed when the patch is pushed.
- This is the 2nd patch in the series of patches to improve the
implementation of the pyosaf utils of OpenSAF services.
- This patch is based on and uses the code change of the
pyosf/
-
revision ba8ecebcd399440496f0baf5bf95e0dcdc9b64fd
Author: Nguyen Luu
Date: Fri, 27 Oct 2017 16:01:03 +0700
pyosaf: High level python interfaces for NTF [#2602]
Improve the implementation of NTF pyosaf utils.
The following comments shall be removed when the patch is pushed.
- This is the 2nd patch in
Improve the implementation of NTF pyosaf utils.
The following comments shall be removed when the patch is pushed.
- This is the 2nd patch in the series of patches to improve the
implementation of the pyosaf utils for OpenSAF services.
- This patch is based on and uses the code change of the
py
-
revision e582f51adac286b9492fcc358468cbc0c7fdf2cb
Author: Nguyen Luu
Date: Fri, 27 Oct 2017 20:52:50 +0700
pyosaf: High level python interfaces for NTF [#2602]
Improve the implementation of NTF pyosaf utils
Added Files:
python/pyosaf/utils/ntf/agent.py
python/pyosaf/utils/ntf/producer.
Improve the implementation of NTF pyosaf utils
---
python/pyosaf/saNtf.py| 4 +-
python/pyosaf/utils/ntf/__init__.py | 855 ++-
python/pyosaf/utils/ntf/agent.py | 500 ++
python/pyosaf/utils/ntf/producer.py | 711 ++
-
revision 6b05b92f072e0f247b52c656600ceaba0a852d86
Author: Nguyen Luu
Date: Fri, 27 Oct 2017 21:53:04 +0700
pyosaf: High level python interfaces for NTF [#2602]
Improve the implementation of NTF pyosaf utils
Added Files:
python/pyosaf/utils/ntf/agent.py
python/pyosaf/utils/ntf/producer.
Improve the implementation of NTF pyosaf utils
---
python/Makefile.am| 6 +-
python/pyosaf/saNtf.py| 4 +-
python/pyosaf/utils/ntf/__init__.py | 855 ++-
python/pyosaf/utils/ntf/agent.py | 500 ++
python/pyo
Fix pylint and pep8 issues for the following Python files:
./tools/devel/dot/trace2dot
./tools/devel/review/patch-tokenize.py
./src/imm/tools/immxml-merge
./src/imm/tools/immxml-validate
./src/imm/tools/baseimm.py
---
src/imm/tools/baseimm.py | 376 ---
src/imm/to
-
revision bcbf709cdd7a35ce0dfa7e2278092d2924740255
Author: Nguyen Luu
Date: Tue, 31 Oct 2017 17:14:49 +0700
tools: Fix pylint and pep8 issues for tools written in Python [#2664]
Fix pylint and pep8 issues for the following Python files:
./tools/devel/dot/trace2dot
./tools/devel/review/patch-tokenize.
Hi Hans,
Ack from me. Tested OK with both pylint-1.6.5 and pylint-1.7.4.
I just have one minor suggestion for the README. Please find it inline,
marked with [Nguyen].
Thanks,
Nguyen
On 11/17/2017 6:50 PM, Hans Nordeback wrote:
The output format can now be given, e.g.:
make pylint (defaults
2: updated with pylint and pep8 corrections for the change added by
ticket #2663 ***
revision aad75ae26682701562f01b090ad7fdbe999a0746
Author: Nguyen Luu
Date: Mon, 20 Nov 2017 17:51:19 +0700
tools: Fix pylint and pep8 issues for tools written in Python [#2664]
Fix pylint and pep8 issues for the
Fix pylint and pep8 issues for the following Python files:
./tools/devel/dot/trace2dot
./tools/devel/review/patch-tokenize.py
./src/imm/tools/immxml-merge
./src/imm/tools/immxml-validate
./src/imm/tools/baseimm.py
---
src/imm/tools/baseimm.py | 376 ---
src/imm/to
-
revision 17e0c8f44cb70a9763ef8b5b825f20b84d76627b
Author: Nguyen Luu
Date: Tue, 5 Dec 2017 13:52:39 +0700
imm: Restore printout format in immxml tools to maintain backward compatibility
[#2728]
- Restore the previous printout format in the immxml tools, which
happened to be changed by ticket
- Restore the previous printout format in the immxml tools, which
happened to be changed by ticket #2664 for better readability, in order
to maintain backward-compatibility for any user scripts/tools that have
been depending their check on the former printout.
- Some other minor updates regar
-
revision 17e0c8f44cb70a9763ef8b5b825f20b84d76627b
Author: Nguyen Luu
Date: Tue, 5 Dec 2017 13:52:39 +0700
imm: Restore printout format in immxml tools to maintain backward compatibility
[#2728]
- Restore the previous printout format in the immxml tools, which
happened to be changed by ticket
- Restore the previous printout format in the immxml tools, which
happened to be changed by ticket #2664 for better readability, in order
to maintain backward-compatibility for any user scripts/tools that have
been depending their check on the former printout.
- Some other minor updates regar
print_info_stderr("Note! Merge ignored %d objects "
- "(parsed: %d stored: %d)", diff_objects,
+ "(parsed:%d stored:%d)", diff_objects,
self.objects_parsed, len(self.objectList)
Thanks,
Ng
agent.py:
- Update some default values for notificationSend.
- Correct NtfAgent.finalize() to handle BAD_HANDLE error code.
producer.py:
- Fix type conversion when assigning value of type SaNtfValueT.
- Keep the notification info for reuse after sending notification.
- Update the assignin
e52
Author: Nguyen Luu
Date: Wed, 20 Dec 2017 14:14:11 +0700
pyosaf: Fix some issues in NTF utils [#2682]
agent.py:
- Update some default values for notificationSend.
- Correct NtfAgent.finalize() to handle BAD_HANDLE error code.
producer.py:
- Fix type conversion when assigning val
-
revision fcec6c823a95bd8cf397edbe80600f3a129f6a87
Author: Nguyen Luu
Date: Wed, 20 Dec 2017 15:18:40 +0700
pyosaf: Fix some issues in NTF utils [#2682]
agent.py:
- Update some default values for notificationSend.
- Correct NtfAgent.finalize() to handle BAD_HANDLE error code.
producer.py:
- Fix type conversion when assignin
agent.py:
- Update some default values for notificationSend.
- Correct NtfAgent.finalize() to handle BAD_HANDLE error code.
producer.py:
- Fix type conversion when assigning value of type SaNtfValueT.
- Keep the notification info for reuse after sending notification.
- Update the assignin
-
revision d56df110afd51411cb4fa75f6655ccca171e9f89
Author: Nguyen Luu
Date: Wed, 20 Dec 2017 15:23:32 +0700
pyosaf: Refactor NTF samples to make use of new pyosaf utils [#2707]
- Refactor the ntfsend and ntfsubscribe python samples to make use of
the new pyosaf utils implementation of enhancement ticket #2602.
-
- Refactor the ntfsend and ntfsubscribe python samples to make use of
the new pyosaf utils implementation of enhancement ticket #2602.
- Add a new ntfread sample to demonstrate usage of the new NtfReader
utils.
---
python/samples/ntfread | 411
python/samples/ntfsend
Valid environment variable should have the format 'var=value'.
AMF currently does not validate this format during CREATE CCBs
for comptype and comp objects (MODIFY allowed for comp after #2255)
related to saAmfxxxCmdEnv attribute.
Besides, the existing validation in avnd_comp_clc_cmd_execute()
is
uot; above):
-
revision bf0660bdc4010f77086dabb6ea2c5b0a77ca4070
Author: Nguyen Luu
Date: Tue, 6 Feb 2018 09:44:41 +0700
amf: validate env variable format set in comptype/comp objects [#2409]
Valid environment variable should have the format 'var=value'.
AM
uot; above):
-
revision bf0660bdc4010f77086dabb6ea2c5b0a77ca4070
Author: Nguyen Luu
Date: Tue, 6 Feb 2018 09:44:41 +0700
amf: validate env variable format set in comptype/comp objects [#2409]
Valid environment variable should have the format 'var=value'.
AM
Valid environment variable should have the format 'var=value'.
AMF currently does not validate this format during CREATE CCBs
for comptype and comp objects (MODIFY allowed for comp after #2255)
related to saAmfxxxCmdEnv attribute.
Besides, the existing validation in avnd_comp_clc_cmd_execute()
is
Hi Gary,
Thank you for your comments. Please see my corresponding answers below,
marked with [Nguyen].
Thanks,
Nguyen
On 2/7/2018 11:30 AM, Gary Lee wrote:
Hi Nguyen
Some comments below marked with [GL].
Thanks
Gary
On 6/2/18, 6:35 pm, "Nguyen Luu" wrote:
Valid e
-
revision de870ceadd4d59cacbacfad3cfed29b6e5fec615
Author: Nguyen Luu
Date: Tue, 13 Feb 2018 10:51:49 +0700
amf: Validate env variable format set in comptype/comp objects [#2409]
Valid environment variable should have the format 'var=value'.
This validation shall now be done by AMF during CCB operati
Valid environment variable should have the format 'var=value'.
This validation shall now be done by AMF during CCB operations
on SaAmfCompType (CREATE) and SaAmfComp (CREATE/MODIFY) objects
regarding the saAmfxxxCmdEnv attribute.
---
src/amf/amfd/comp.cc | 59 ++
-
revision cbe183d7927cd53d6928611b91f5fcda5d939eee
Author: Nguyen Luu
Date: Tue, 6 Mar 2018 13:14:52 +0700
amfa: Fix api internal check to avoid fatal mutex unlock [#2548]
Current check for the required setting of the SA_AMF_COMPONENT_NAME
env variable in some amf api's (ComponentRegister, QuiescingComplete)
Current check for the required setting of the SA_AMF_COMPONENT_NAME
env variable in some amf api's (ComponentRegister, QuiescingComplete)
would crash the invoking process if that env variable was missed
to be set for some reason, as the agent lib tries, during cleanup,
to unlock a mutex which it ha
Hi Hans,
Please help me review this patch whenever possible for you. It has been
on review for months.
Thanks,
Nguyen
On 12/20/2017 3:31 PM, Nguyen Luu wrote:
- Refactor the ntfsend and ntfsubscribe python samples to make use of
the new pyosaf utils implementation of enhancement ticket
39 AM, Nguyen Luu wrote:
Current check for the required setting of the SA_AMF_COMPONENT_NAME
env variable in some amf api's (ComponentRegister, QuiescingComplete)
would crash the invoking process if that env variable was missed
to be set for some reason, as the agent lib tries, during cleanup,
t
Current handling of SMFND DOWN event does not take into account failed
SMFND UP event, which could eventually result in an inexact view of the
actual number of SMFND nodes in the cluster if, for example, a node
happened to be DOWN and UP twice, and the first UP event somehow failed.
---
src/smf/sm
uot; above):
-
revision 218f71b784413703352d618fc9e657a087947635
Author: Nguyen Luu
Date: Wed, 28 Mar 2018 13:51:47 +0700
smfd: Fix incorrect handling of SMFND NCSMDS_UP/DOWN events [#2821]
Current handling of SMFND DOWN event does not take into account failed
SMFND UP event, w
Hi Lennart, Vijay,
Please help me review this patch whenever possible for you.
Thanks,
Nguyen
On 3/28/2018 3:07 PM, Nguyen Luu wrote:
Current handling of SMFND DOWN event does not take into account failed
SMFND UP event, which could eventually result in an inexact view of the
actual number of
ote:
Hi Nguyen,
Ack with comments. See below [Lennart]
Thanks
Lennart
-Original Message-
From: Nguyen Luu [mailto:nguyen.tk@dektech.com.au]
Sent: den 28 mars 2018 10:08
To: Lennart Lund ; vijay@oracle.com
Cc: opensaf-devel@lists.sourceforge.net; Nguyen Tran Khoi Luu
Subject:
Hi,
I intend to push this patch by the end of today if there are no more
comments.
Thanks,
Nguyen
On 4/6/2018 10:54 AM, Nguyen Luu wrote:
Hi Lennart,
Thank you for your comment.
You suggested that the changed files be reformatted following Google
Coding Style (i.e space-indented); but
-
revision 99484e9ba2fce28b997cfd6a0af571c99c876804
Author: Nguyen Luu
Date: Thu, 3 May 2018 09:47:12 +0700
smfd: Remove duplicate DU/AU on SU/comp level in one-step upgrade [#2227]
This fix extends the previous one of #2209 which was on node level.
In particular, it is to eliminate any duplicate DU/AU on SU or
This fix extends the previous one of #2209 which was on node level.
In particular, it is to eliminate any duplicate DU/AU on SU or component
level when merging forAddRemove and forModify/rolling procedures into a
single-step procedure for one-step upgrade execution mode.
The fix also corrects retur
-
revision dae399f6d30c7cac041282d8d3c5510838fdc3cd
Author: Nguyen Luu
Date: Tue, 8 May 2018 08:57:12 +0700
smfd: Remove duplicate DU/AU on SU/comp level in one-step upgrade [#2227]
This fix extends the previous one of #2209 which was on node level.
In particular, it is to eliminate any duplicate DU/AU on SU or
This fix extends the previous one of #2209 which was on node level.
In particular, it is to eliminate any duplicate DU/AU on SU or component
level when merging forAddRemove and forModify/rolling procedures into a
single-step procedure for one-step upgrade execution mode.
---
src/smf/smfd/SmfUpgrad
Lennart
-Original Message-
From: Nguyen Luu [mailto:nguyen.tk@dektech.com.au]
Sent: den 8 maj 2018 04:19
To: Lennart Lund ; syam.tall...@oracle.com
Cc: opensaf-devel@lists.sourceforge.net; Nguyen Tran Khoi Luu
Subject: [PATCH 0/1] Review Request for smfd: Remove duplicate DU, AU on
SU
This fix extends, and also improves the previous one of #2209 which was
done for node level DU/AU. In particular, it eliminates any duplicate AU/DU
on SU and component level when merging forAddRemove and forModify/Rolling
procedures into a single-step procedure for one-step upgrade execution mode.
168ea43f78
Author: Nguyen Luu
Date: Sun, 13 May 2018 11:37:53 +0700
smfd: Remove duplicate DU/AU on SU/comp level in one-step upgrade [#2227]
This fix extends, and also improves the previous one of #2209 which was
done for node level DU/AU. In particular, it eliminates any duplicate AU/DU
functions according to
Google style guide so there may be a mix in some classes. However this file is
fixed.
SmfUtils.cc:
lldtest_delete is test code that should have been removed. Fixed
Thanks
Lennart
-Original Message-
From: Nguyen Luu [mailto:nguyen.tk@dektech.com.au]
Sent: d
Hi Lennart,
I've reviewed and tested your patch.
- For the code review, please check comments in the attached diff file.
- For the testing, a core dump occurred when executing *test_ccbhdl*
(always reproducible). Test printouts, core dump backtrace, and syslog
as shown below. Looks like some is
,
Attached is a diff with your comments still in there and my answers
tagged [Lennart].
I have also kept your uncommented fixes.
If you ack this I will use the patch with review comments removed.
Thanks
Lennart
*From:*Nguyen Luu
*Sent:* den 31 maj 2018 11:10
*To:* Lennart Lund
*Cc:* opensaf-devel
l happen in a controlled
and predictable way.
What kind of system are you using?
Thanks
Lennart
*From:*Nguyen Luu
*Sent:* den 31 maj 2018 11:10
*To:* Lennart Lund
*Cc:* opensaf-devel@lists.sourceforge.net
*Subject:* Re: [PATCH 0/1] Review Request for smf: Validation error
for rollback
Hi Lennart,
Ack from me for this new patch. Code review only, no re-test.
Just a minor comment below for you to consider.
Since we will now not add the CreateDescriptor for an object to the CCB
if it already exists (CheckObjectExist::IsExisting() ==
CheckObjectExist::ReturnCode::kOk), do we s
s that has to be deleted).
Thanks
Lennart
*From:*Nguyen Luu
*Sent:* den 7 juni 2018 07:19
*To:* Lennart Lund
*Cc:* opensaf-devel@lists.sourceforge.net
*Subject:* Re: [PATCH 1/1] smf: Validation error for rollback CCB and
a related core dump [#2858]
Hi Lennart,
Ack from me for this new p
Hi Lennart,
Looks like you have by mistake commented on the wrong mail thread.
Thanks for the comments and suggestion anyway. I will consider them and
work on a new fix.
Thanks,
Nguyen
On 6/19/2018 9:30 PM, Lennart Lund wrote:
Hi Nguyen
The main issue that I saw is that you have added some
Currently, if an upgrade DELETE or MODIFY CCB contains empty-value
attributes, SMF will create rollback data for those attributes with
the hard-coded "<_empty_>" string value. However, this hard-coded
string will actually make the later rollback fail, especially for
numeric-type attributes, when tr
uot; above):
-
revision 6a3ea7621b35c4562571f8dd6156ad03c1cd8a2e
Author: Nguyen Luu
Date: Wed, 20 Jun 2018 17:30:42 +0700
smf: Fix failed rollback of DELETE/MODIFY CCB with originally empty attributes
[#2877]
Currently, if an upgrade DELETE or MODIFY CCB contains empty-value
attri
Currently, if an upgrade DELETE or MODIFY CCB contains empty-value
attributes, SMF will create rollback data for those attributes with
the hard-coded "<_empty_>" string value. However, this hard-coded
string will actually make the later rollback fail, especially for
numeric-type attributes, when tr
uot; above):
-
revision 3d1d6e45cd49fabddcdab00ad998af7583090a20
Author: Nguyen Luu
Date: Thu, 21 Jun 2018 19:46:55 +0700
smf: Fix failed rollback of DELETE/MODIFY CCB with originally empty attributes
[#2877]
Currently, if an upgrade DELETE or MODIFY CCB contains empty-value
attri
Hi,
One comment from me for the patch: With the new flow, the matched hdl
should be removed from the cbk_list if posting the message to the client
mailbox fails.
A general comment: The original issue happened because of a race
condition in which the smfa_cb was used in different threads with
ink the posting fail is critical problem since agent will miss a callback.
Also avoid smfnd is keeping waiting for agent response for that callback.
Please correct if I was wrong.
Best Regards,
Thuan (UFO – Unique FBI Opensaf)
CoreMW Maintenance, DEK VietNam
-Original Message-----
From: Nguyen
thread of SMFD, it crash mean SMFD crash (node reboot?), is there
any rollback trigger after that?
If it cannot, then I should handle posting fail as your comment.
Best Regards,
Thuan
-Original Message-----
From: Nguyen Luu
Sent: Thursday, July 5, 2018 8:00 PM
To: Tran Thuan ;
rds,
Thuan (UFO – Unique FBI Opensaf)
CoreMW Maintenance, DEK VietNam
-Original Message-----
From: Nguyen Luu
Sent: Friday, July 6, 2018 5:34 PM
To: Tran Thuan ; lennart.l...@ericsson.com; 'Gary Lee'
Cc: opensaf-devel@lists.sourceforge.net
Subject: Re: [devel] [PATCH 1/1] smf: Incre
Hi Thuan,
I also agree with Gary about removing assert(). Other changes look fine.
Thanks,
Nguyen
On 7/11/2018 12:52 PM, Gary Lee wrote:
Hi Thuan
Looks OK to me, but I would remove the assert. We shouldn't do that in library
code.
Thanks
Gary
On 10/7/18, 6:52 pm, "thuan.tran" wrote:
Hi Thuan,
Ack from me.
Just a suggestion for more correct comment (in case there's only one
handle matching the callback label):
/* Descrease the cbk count */ => /* Remove the agent hdl from the cbk_list */
Thanks,
Nguyen
On 7/13/2018 10:52 AM, thuan.tran wrote:
Sometimes, callback agent d
96 matches
Mail list logo