[tickets] [opensaf:tickets] #1938 AMFD: Incorrect setting SU readiness state
- **status**: review --> fixed - **assigned_to**: Minh Hon Chau --> nobody --- ** [tickets:#1938] AMFD: Incorrect setting SU readiness state** **Status:** fixed **Milestone:** 5.0.1 **Created:** Mon Aug 08, 2016 08:01 AM UTC by Minh Hon Chau **Last Updated:** Tue Aug 09, 2016 12:22 PM UTC **Owner:** nobody In SU repair scenario, the SU's readiness state is currently set to IN_SERVICE in avd_data_update_req_evh() case saAmfSUOperState_ID: TRACE("oper pres state"); if (n2d_msg->msg_info.n2d_data_req.param_info.value_len == sizeof(uint32_t)) { l_val = ntohl(*((uint32_t *)&n2d_msg->msg_info.n2d_data_req.param_info.value[0])); su->set_oper_state(l_val); } if (su->is_in_service() == true) { su->set_readiness_state(SA_AMF_READINESS_IN_SERVICE); } And later on, avd_su_oper_state_evh() set SU's readiness state back to OUT_OF_SERVICE. if (avd_cb->scs_absence_max_duration > 0 && su->saAmfSUPresenceState == SA_AMF_PRESENCE_UNINSTANTIATED && su->saAmfSUPreInstantiable == false) { // this is to allow non NPI SUs to be repaired if // headless mode is enabled. Otherwise, the code // following will assume the SU is already in service // when it failed to instantiate while headless if (cb->init_state == AVD_APP_STATE) { LOG_NO("Setting NPI SU '%s' to OOS after headless state", su->name.value); su->set_readiness_state(SA_AMF_READINESS_OUT_OF_SERVICE); } } This logic is redundant and SU's readiness state should not be set while updating SU' oper state to align with non-headless's logic consistency --- Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is subscribed to https://sourceforge.net/p/opensaf/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- ___ Opensaf-tickets mailing list Opensaf-tickets@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
[tickets] [opensaf:tickets] #1938 AMFD: Incorrect setting SU readiness state
- **status**: assigned --> review --- ** [tickets:#1938] AMFD: Incorrect setting SU readiness state** **Status:** review **Milestone:** 5.0.1 **Created:** Mon Aug 08, 2016 08:01 AM UTC by Minh Hon Chau **Last Updated:** Mon Aug 08, 2016 08:01 AM UTC **Owner:** Minh Hon Chau In SU repair scenario, the SU's readiness state is currently set to IN_SERVICE in avd_data_update_req_evh() case saAmfSUOperState_ID: TRACE("oper pres state"); if (n2d_msg->msg_info.n2d_data_req.param_info.value_len == sizeof(uint32_t)) { l_val = ntohl(*((uint32_t *)&n2d_msg->msg_info.n2d_data_req.param_info.value[0])); su->set_oper_state(l_val); } if (su->is_in_service() == true) { su->set_readiness_state(SA_AMF_READINESS_IN_SERVICE); } And later on, avd_su_oper_state_evh() set SU's readiness state back to OUT_OF_SERVICE. if (avd_cb->scs_absence_max_duration > 0 && su->saAmfSUPresenceState == SA_AMF_PRESENCE_UNINSTANTIATED && su->saAmfSUPreInstantiable == false) { // this is to allow non NPI SUs to be repaired if // headless mode is enabled. Otherwise, the code // following will assume the SU is already in service // when it failed to instantiate while headless if (cb->init_state == AVD_APP_STATE) { LOG_NO("Setting NPI SU '%s' to OOS after headless state", su->name.value); su->set_readiness_state(SA_AMF_READINESS_OUT_OF_SERVICE); } } This logic is redundant and SU's readiness state should not be set while updating SU' oper state to align with non-headless's logic consistency --- Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is subscribed to https://sourceforge.net/p/opensaf/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports. http://sdm.link/zohodev2dev___ Opensaf-tickets mailing list Opensaf-tickets@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
[tickets] [opensaf:tickets] #1938 AMFD: Incorrect setting SU readiness state
--- ** [tickets:#1938] AMFD: Incorrect setting SU readiness state** **Status:** assigned **Milestone:** 5.0.1 **Created:** Mon Aug 08, 2016 08:01 AM UTC by Minh Hon Chau **Last Updated:** Mon Aug 08, 2016 08:01 AM UTC **Owner:** Minh Hon Chau In SU repair scenario, the SU's readiness state is currently set to IN_SERVICE in avd_data_update_req_evh() case saAmfSUOperState_ID: TRACE("oper pres state"); if (n2d_msg->msg_info.n2d_data_req.param_info.value_len == sizeof(uint32_t)) { l_val = ntohl(*((uint32_t *)&n2d_msg->msg_info.n2d_data_req.param_info.value[0])); su->set_oper_state(l_val); } if (su->is_in_service() == true) { su->set_readiness_state(SA_AMF_READINESS_IN_SERVICE); } And later on, avd_su_oper_state_evh() set SU's readiness state back to OUT_OF_SERVICE. if (avd_cb->scs_absence_max_duration > 0 && su->saAmfSUPresenceState == SA_AMF_PRESENCE_UNINSTANTIATED && su->saAmfSUPreInstantiable == false) { // this is to allow non NPI SUs to be repaired if // headless mode is enabled. Otherwise, the code // following will assume the SU is already in service // when it failed to instantiate while headless if (cb->init_state == AVD_APP_STATE) { LOG_NO("Setting NPI SU '%s' to OOS after headless state", su->name.value); su->set_readiness_state(SA_AMF_READINESS_OUT_OF_SERVICE); } } This logic is redundant and SU's readiness state should not be set while updating SU' oper state to align with non-headless's logic consistency --- Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is subscribed to https://sourceforge.net/p/opensaf/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- ___ Opensaf-tickets mailing list Opensaf-tickets@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-tickets