Hi Praveen,
Thanks for your information. I will recheck it.
Best regards,
Long Nguyen.
On 11/10/2016 11:18 AM, praveen malviya wrote:
> Hi,
> Thanks for the information.
> I think, like #2175, this issue may not occur after #2112 being
> pushed. Can you please recheck?
>
> Thanks,
> Praveen
>
> On 10-Nov-16 9:38 AM, Long Nguyen wrote:
>> Hi Praveen,
>>
>> Yes, I think the PL-6 was added after avd_node_config_get() is called so
>> SC-2 did not have the info about PL-6.
>>
>> Best regards,
>> Long Nguyen.
>>
>> On 11/8/2016 6:50 PM, praveen malviya wrote:
>>> Hi Long,
>>>
>>> I have gone through the traces from both the AMFDs. I think there is a
>>> time difference between the controllers. I just wanted to see whether
>>> SC-2 got any CCBs for PL-6 addition. At the same time SC-2 did not get
>>> PL-6 information in avd_node_config_get() also.
>>> I guess PL-6 was added when SC-2 completed at-least reading
>>> node_configuration and has not done applier set.
>>> Is this the state of SC-2 when PL-6 was added?
>>>
>>>
>>> Thanks,
>>> Praveen
>>>
>>> On 02-Nov-16 9:28 AM, Long HB Nguyen wrote:
>>>> osaf/services/saf/amf/amfd/ckpt_updt.cc | 14 +++++++++-----
>>>> 1 files changed, 9 insertions(+), 5 deletions(-)
>>>>
>>>>
>>>> diff --git a/osaf/services/saf/amf/amfd/ckpt_updt.cc
>>>> b/osaf/services/saf/amf/amfd/ckpt_updt.cc
>>>> --- a/osaf/services/saf/amf/amfd/ckpt_updt.cc
>>>> +++ b/osaf/services/saf/amf/amfd/ckpt_updt.cc
>>>> @@ -20,6 +20,7 @@
>>>> #include <sg.h>
>>>> #include <csi.h>
>>>> #include <app.h>
>>>> +#include <cluster.h>
>>>>
>>>> static char *action_name[] = {
>>>> const_cast<char*>("invalid"),
>>>> @@ -53,10 +54,14 @@ uint32_t avd_ckpt_node(AVD_CL_CB *cb, AV
>>>>
>>>> osafassert (action == NCS_MBCSV_ACT_UPDATE);
>>>>
>>>> - if (nullptr == (node = avd_node_get(ckpt_node->name))) {
>>>> - LOG_WA("avd_node_get FAILED for '%s'",
>>>> ckpt_node->name.c_str());
>>>> - rc = NCSCC_RC_FAILURE;
>>>> - goto done;
>>>> + node = avd_node_get(ckpt_node->name);
>>>> + if (node == nullptr) {
>>>> + TRACE("'%s' does not exist, creating it",
>>>> ckpt_node->name.c_str());
>>>> + node = avd_node_new(ckpt_node->name);
>>>> + node->cluster = avd_cluster;
>>>> + node->admin_ng = nullptr;
>>>> + rc = node_name_db->insert(node->name, node);
>>>> + osafassert(rc == NCSCC_RC_SUCCESS);
>>>> }
>>>> /* Update all runtime attributes */
>>>> node->saAmfNodeAdminState = ckpt_node->saAmfNodeAdminState;
>>>> @@ -72,7 +77,6 @@ uint32_t avd_ckpt_node(AVD_CL_CB *cb, AV
>>>> if (nullptr == avd_node_find_nodeid(ckpt_node->node_info.nodeId))
>>>> avd_node_add_nodeid(node);
>>>>
>>>> -done:
>>>> TRACE_LEAVE2("%u", rc);
>>>> return rc;
>>>> }
>>>>
>>>
>>
>
------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
