changeset: 8635:5006daf8130e
user: Tai Dinh <tai.d...@dektech.com.au>
date: Fri Mar 03 13:20:14 2017 +1100
summary: amfd: SG admin state is nor honored during node group
unlock_instantiation [#2324]
changeset: 8636:88d0cebf7bb7
branch: opensaf-5.1.x
parent: 8620:5be5a71b5069
user: Tai Dinh <tai.d...@dektech.com.au>
date: Fri Mar 03 13:21:07 2017 +1100
summary: amfd: SG admin state is nor honored during node group
unlock_instantiation [#2324]
changeset: 8637:c011fbe3fdcb
branch: opensaf-5.0.x
parent: 8619:addb5d754d71
user: Tai Dinh <tai.d...@dektech.com.au>
date: Fri Mar 03 13:25:46 2017 +1100
summary: amfd: SG admin state is nor honored during node group
unlock_instantiation [#2324]
---
** [tickets:#2324] amfd: SG admin state is nor honored during node group
unlock_instantiation**
**Status:** review
**Milestone:** 5.2.RC1
**Created:** Fri Feb 24, 2017 07:20 AM UTC by Tai Dinh
**Last Updated:** Thu Mar 02, 2017 06:01 AM UTC
**Owner:** Tai Dinh
SG admin state is not honored during node group unlock_instantiation.
This can always be reproduced:
- Put the SG under locked_instantiation state. SU is now UNINSTANTIATED
- Lock/lock_in the node group that the SG or its SUs hosted on
- Unlock_in the node group, the expectation here is that SU should be kept as
UNINSTANTIATED state but they're not.
- AMF always try to instantiate the SUs without checking their SG state.
if ((su->saAmfSUAdminState != SA_AMF_ADMIN_LOCKED_INSTANTIATION) &&
(su->su_on_node->saAmfNodeAdminState !=
SA_AMF_ADMIN_LOCKED_INSTANTIATION) &&
(su->saAmfSUOperState == SA_AMF_OPERATIONAL_ENABLED) &&
(su->saAmfSUPresenceState ==
SA_AMF_PRESENCE_UNINSTANTIATED) &&
(su->su_on_node->saAmfNodeOperState ==
SA_AMF_OPERATIONAL_ENABLED)) {
if ((su->saAmfSUPreInstantiable == false) ||
(su->su_on_node->node_state != AVD_AVND_STATE_PRESENT))
continue;
if (sg->saAmfSGNumPrefInserviceSUs > su_try_inst) {
if (avd_snd_presence_msg(avd_cb, su, false) !=
NCSCC_RC_SUCCESS) {
LOG_NO("Failed to send Instantiation of '%s'",
su->name.c_str());
} else {
su->su_on_node->su_cnt_admin_oper++;
su_try_inst++;
}
}
}
---
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.------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-tickets mailing list
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
