Re: [devel] [PATCH 1/1] amfd: Only start clm track for 2N Opensaf SU in failover [#2980]

[Hans Nordebäck]

Hi Minh,

ack, review only/Thanks HansN

On 12/10/18 07:00, Minh Chau wrote:
> ---
>   src/amf/amfd/sg_2n_fsm.cc | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/amf/amfd/sg_2n_fsm.cc b/src/amf/amfd/sg_2n_fsm.cc
> index a218786..91ffc63 100644
> --- a/src/amf/amfd/sg_2n_fsm.cc
> +++ b/src/amf/amfd/sg_2n_fsm.cc
> @@ -1784,7 +1784,8 @@ uint32_t SG_2N::susi_success_sg_realign(AVD_SU *su, 
> AVD_SU_SI_REL *susi,
> }
>   
> if ((state == SA_AMF_HA_ACTIVE) &&
> -  (cb->node_id_avd == su->su_on_node->node_info.nodeId)) {
> +  (cb->node_id_avd == su->su_on_node->node_info.nodeId) &&
> +  (su->sg_of_su->sg_ncs_spec == true)) {
>   /* This is as a result of failover, start CLM tracking*/
>   if (avd_clm_track_start(cb) == SA_AIS_ERR_TRY_AGAIN)
> Fifo::queue(new ClmTrackStart());

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

[devel] [PATCH 1/1] amfd: Only start clm track for 2N Opensaf SU in failover [#2980]

[Minh Chau]

---
 src/amf/amfd/sg_2n_fsm.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/amf/amfd/sg_2n_fsm.cc b/src/amf/amfd/sg_2n_fsm.cc
index a218786..91ffc63 100644
--- a/src/amf/amfd/sg_2n_fsm.cc
+++ b/src/amf/amfd/sg_2n_fsm.cc
@@ -1784,7 +1784,8 @@ uint32_t SG_2N::susi_success_sg_realign(AVD_SU *su, 
AVD_SU_SI_REL *susi,
   }
 
   if ((state == SA_AMF_HA_ACTIVE) &&
-  (cb->node_id_avd == su->su_on_node->node_info.nodeId)) {
+  (cb->node_id_avd == su->su_on_node->node_info.nodeId) &&
+  (su->sg_of_su->sg_ncs_spec == true)) {
 /* This is as a result of failover, start CLM tracking*/
 if (avd_clm_track_start(cb) == SA_AIS_ERR_TRY_AGAIN)
   Fifo::queue(new ClmTrackStart());
-- 
2.7.4



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