HI Nagu

Ack (tested and legacy tests run)

Thanks

-----Original Message-----
From: <[email protected]>
Date: Wednesday, 1 March 2017 at 8:50 pm
To: <[email protected]>, <[email protected]>, 
<[email protected]>, gary <[email protected]>
Cc: <[email protected]>
Subject: [PATCH 1 of 1] amfd: dont create susi when node is absent [#2106]

     src/amf/amfd/su.cc |  4 +++-
     1 files changed, 3 insertions(+), 1 deletions(-)
    
    
    diff --git a/src/amf/amfd/su.cc b/src/amf/amfd/su.cc
    --- a/src/amf/amfd/su.cc
    +++ b/src/amf/amfd/su.cc
    @@ -891,6 +891,7 @@ void AVD_SU::set_admin_state(SaAmfAdminS
     
     void AVD_SU::unlock(SaImmOiHandleT immoi_handle, SaInvocationT invocation) 
{
        bool is_oper_successful = true;
    +   AVD_AVND *avnd = get_node_ptr();
     
        TRACE_ENTER2("'%s'", name.c_str());
        set_admin_state(SA_AMF_ADMIN_UNLOCKED);
    @@ -903,7 +904,8 @@ void AVD_SU::unlock(SaImmOiHandleT immoi
                goto done;
        }
     
    -   if ((is_in_service() == true) || (sg_of_su->sg_ncs_spec == true)) {
    +   if ((is_in_service() == true) || ((sg_of_su->sg_ncs_spec == true) &&
    +                           ((avnd) && (avnd->node_state != 
AVD_AVND_STATE_ABSENT)))) {
                /* Reason for checking for MW component is that node oper state 
and
                 * SU oper state are marked enabled after they gets assignments.
                 * So, we can't check compatibility with is_in_service() for 
them.
    



------------------------------------------------------------------------------
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

Reply via email to