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

Reply via email to