Hi Praveen,

I have tested with the patch for #2112. With the patch for #2112, I am 
not able to reproduce the problem anymore.
I think we can ignore this ticket.

Thanks so much,
Long Nguyen.

On 11/10/2016 11:23 AM, Long Nguyen wrote:
> 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

Reply via email to