changeset: 4615:2e065a4e240f
tag: tip
user:Hans Feldt
date:Wed Nov 20 11:49:16 2013 +0100
summary: amfd: fix admin op logging [#85]
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** fixed
**Created:** Mon May 13, 2013 04:44 AM UTC by N
- **status**: review --> fixed
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** fixed
**Created:** Mon May 13, 2013 04:44 AM UTC by Nagendra Kumar
**Last Updated:** Thu Oct 10, 2013 11:08 AM UTC
**Owner:** Nagendra Kumar
Migrated from http://devel.opensaf.org/tick
changeset: 4614:70f20d16c9a8
tag: tip
user:Nagendra Kumar
date:Wed Nov 20 11:03:05 2013 +0530
summary: amfd: Add support for admin op result strings [#85]
[staging:70f20d]
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** fixed
**Creat
- **status**: accepted --> review
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** review
**Created:** Mon May 13, 2013 04:44 AM UTC by Nagendra Kumar
**Last Updated:** Tue Oct 08, 2013 04:38 AM UTC
**Owner:** Nagendra Kumar
Migrated from http://devel.opensaf.org/
In case of SA_AIS_ERR_NOT_SUPPORTED, practically, we need to inform user about
our proprietary implementation. For example:
if ((su->sg_of_su->sg_ncs_spec == SA_TRUE)
&& (cb->node_id_avd == su->su_on_node->node_info.nodeId)) {
rc = SA_AIS_ERR_NOT_SUPPORTED;
I think we should create new utility functions. Function 1:
void immAdminOperationResult(SaInvocationT invocation,
SaAisErrorT result,
const char *format, ...);
the purpose of this one is to be used upfront at
And to mention, we should take care and design those returned string much more
carefully. Consider some examples:
$ rm test
rm: cannot remove `test': No such file or directory
$ mkdir /test
mkdir: cannot create directory `/test': Permission denied
---
** [tickets:#85] AMF: admin ops should re
Hi Hans,
There has been LOG_ER, LOG_WA and LOG_NO in case admin op failures.
Does this ticket was raised to eliminate all the LOG_ER, LOG_WA and LOG_NO and
delete them and use strings in SaImmAdminOperationError. There has been 100's
of places this change will affect. Are we all in sync?
- **status**: assigned --> accepted
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** accepted
**Created:** Mon May 13, 2013 04:44 AM UTC by Nagendra Kumar
**Last Updated:** Fri Sep 06, 2013 12:35 PM UTC
**Owner:** Nagendra Kumar
Migrated from http://devel.opensaf.
- **status**: unassigned --> assigned
- **assigned_to**: Nagendra Kumar
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** assigned
**Created:** Mon May 13, 2013 04:44 AM UTC by Nagendra Kumar
**Last Updated:** Fri Sep 06, 2013 12:34 PM UTC
**Owner:** Nagendra Kumar
- **Milestone**: future --> 4.4.FC
---
** [tickets:#85] AMF: admin ops should return error strings**
**Status:** unassigned
**Created:** Mon May 13, 2013 04:44 AM UTC by Nagendra Kumar
**Last Updated:** Mon May 13, 2013 04:44 AM UTC
**Owner:** nobody
Migrated from http://devel.opensaf.org/tic
11 matches
Mail list logo