Ack with minor comment.
Update commit msg due to the fix just apply to HC period.
-----Original Message-----
From: Dat Tran Quoc Phan <dat.tq.p...@dektech.com.au>
Sent: Friday, December 17, 2021 9:29 AM
To: Hieu Hong Hoang <hieu.h.ho...@dektech.com.au>; Huu The Truong
<huu.t.tru...@dektech.com.au>; Minh Hon Chau <minh.c...@dektech.com.au>; Thang
Duc Nguyen <thang.d.ngu...@dektech.com.au>; Thien Minh Huynh
<thien.m.hu...@dektech.com.au>
Cc: opensaf-devel@lists.sourceforge.net; Dat Tran Quoc Phan
<dat.tq.p...@dektech.com.au>
Subject: [PATCH 1/1] amf: Restart healthcheck timer after changing
saAmfHctDefPeriod/saAmfHctDefMaxDuration [#3298]
Change of saAmfHctDefPeriod/saAmfHctDefMaxDuration will not take effect
immediately.
The healthcheck timer will wait to end the old values of two attributes before
apply new values.
Restart the healthcheck timer right after update new values to make them take
effect immediately.
---
src/amf/amfnd/hcdb.cc | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/src/amf/amfnd/hcdb.cc b/src/amf/amfnd/hcdb.cc index
a99e6904f..aefacf19e 100644
--- a/src/amf/amfnd/hcdb.cc
+++ b/src/amf/amfnd/hcdb.cc
@@ -322,6 +322,11 @@ static void comp_hctype_update_compdb(AVND_CB *cb,
AVSV_PARAM_INFO *param) {
osafassert(0);
break;
}
+ // Ref #3298
+ uint32_t rc = avnd_start_tmr(avnd_cb, &comp_hc_rec->tmr, AVND_TMR_HC,
+ comp_hc_rec->period, comp_hc_rec->opq_hdl);
+ osafassert(rc == NCSCC_RC_SUCCESS);
+ comp_hc_rec->status = AVND_COMP_HC_STATUS_STABLE;
}
}
osaf_extended_name_free(&hlt_chk.comp_name);
--
2.17.1
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
