ack, code review only/Thanks HansN

On 09/19/2018 12:50 AM, Minh Chau wrote:
---
  src/amf/amfd/sg_nwayact_fsm.cc | 8 +++-----
  1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/amf/amfd/sg_nwayact_fsm.cc b/src/amf/amfd/sg_nwayact_fsm.cc
index f339026..bbd63c8 100644
--- a/src/amf/amfd/sg_nwayact_fsm.cc
+++ b/src/amf/amfd/sg_nwayact_fsm.cc
@@ -77,17 +77,15 @@ AVD_SU *avd_sg_nacvred_su_chose_asgn(AVD_CL_CB *cb, AVD_SG 
*sg) {
        continue;
      }
      /* After split-brain, the assignments can go over the preferred number.
-     * Reboot the node once at time until the current active assignments
+     * Remove assignment once at time until the current active assignments
       * is equal to preferred number
       */
      if (i_si->pref_active_assignments() < i_si->curr_active_assignments()) {
        AVD_SU* over_assigned_su = i_si->list_of_sisu->su;
        if (over_assigned_su) {
-        LOG_ER("SU '%s' has been over-assigned",
+        LOG_WA("SU '%s' has been over-assigned, remove its assignment",
              over_assigned_su->name.c_str());
-        LOG_EM("Sending node reboot order to '%s'",
-           over_assigned_su->su_on_node->name.c_str());
-        avd_d2n_reboot_snd(over_assigned_su->su_on_node);
+        sg->su_fault(cb, over_assigned_su);
        }
        goto done;
      }



_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to