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