In that case please mark the ticket invalid with milestone "never".
Thanks
Praveen
On 14-Nov-16 3:59 PM, Long Nguyen wrote:
> 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
