Hi Long, I had already acked it on 14th Oct.
Thanks, Praveen On 18-Oct-16 12:28 PM, Long Nguyen wrote: > Hi Praveen and Nagu, > > Did you have a chance to look at the patch? > > Best regards, > Long Nguyen. > > On 10/10/2016 11:27 AM, Gary Lee wrote: >> Hi Long >> >> ack >> >> Thanks >> >>> On 10 Oct. 2016, at 3:26 pm, Long HB Nguyen >>> <[email protected]> wrote: >>> >>> osaf/services/saf/amf/amfd/sg_2n_fsm.cc | 10 +++++----- >>> 1 files changed, 5 insertions(+), 5 deletions(-) >>> >>> >>> diff --git a/osaf/services/saf/amf/amfd/sg_2n_fsm.cc >>> b/osaf/services/saf/amf/amfd/sg_2n_fsm.cc >>> --- a/osaf/services/saf/amf/amfd/sg_2n_fsm.cc >>> +++ b/osaf/services/saf/amf/amfd/sg_2n_fsm.cc >>> @@ -745,21 +745,21 @@ SaAisErrorT SG_2N::si_swap(AVD_SI *si, S >>> TRACE_ENTER2("'%s' sg_fsm_state=%u", si->name.c_str(), >>> si->sg_of_si->sg_fsm_state); >>> >>> if (si->saAmfSIAdminState != SA_AMF_ADMIN_UNLOCKED) { >>> - LOG_ER("%s SWAP failed - wrong admin state=%u", >>> si->name.c_str(), >>> + LOG_NO("%s SWAP failed - wrong admin state=%u", >>> si->name.c_str(), >>> si->saAmfSIAdminState); >>> rc = SA_AIS_ERR_TRY_AGAIN; >>> goto done; >>> } >>> >>> if (avd_cb->init_state != AVD_APP_STATE) { >>> - LOG_ER("%s SWAP failed - not in app state (%u)", >>> si->name.c_str(), >>> + LOG_NO("%s SWAP failed - not in app state (%u)", >>> si->name.c_str(), >>> avd_cb->init_state); >>> rc = SA_AIS_ERR_TRY_AGAIN; >>> goto done; >>> } >>> >>> if (si->sg_of_si->sg_fsm_state != AVD_SG_FSM_STABLE) { >>> - LOG_ER("%s SWAP failed - SG not stable (%u)", si->name.c_str(), >>> + LOG_NO("%s SWAP failed - SG not stable (%u)", si->name.c_str(), >>> si->sg_of_si->sg_fsm_state); >>> rc = SA_AIS_ERR_TRY_AGAIN; >>> goto done; >>> @@ -774,14 +774,14 @@ SaAisErrorT SG_2N::si_swap(AVD_SI *si, S >>> if ((si->sg_of_si->sg_ncs_spec) && >>> ((avd_cb->node_id_avd_other != 0) && >>> (avd_cb->other_avd_adest != 0))) { >>> if (avd_cb->stby_sync_state == AVD_STBY_OUT_OF_SYNC) { >>> - LOG_ER("%s SWAP failed - Cold sync in progress", >>> si->name.c_str()); >>> + LOG_NO("%s SWAP failed - Cold sync in progress", >>> si->name.c_str()); >>> rc = SA_AIS_ERR_TRY_AGAIN; >>> goto done; >>> } >>> } >>> >>> if (si->list_of_sisu->si_next == nullptr) { >>> - LOG_ER("%s SWAP failed - only one assignment", >>> si->name.c_str()); >>> + LOG_NO("%s SWAP failed - only one assignment", >>> si->name.c_str()); >>> rc = SA_AIS_ERR_TRY_AGAIN; >>> goto done; >>> } >> > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
