Nack. Applier's Validations are necessary as done in ccb_completed_modify_hdlr() of hlt.cc.
Thanks -Nagu > -----Original Message----- > From: Hans Feldt [mailto:[email protected]] > Sent: 07 January 2014 02:25 > To: Nagendra Kumar > Cc: [email protected] > Subject: [PATCH 2 of 2] amfd: allow changes to hctype [#539] > > osaf/services/saf/amf/amfd/hlttype.cc | 6 +++++- > 1 files changed, 5 insertions(+), 1 deletions(-) > > > diff --git a/osaf/services/saf/amf/amfd/hlttype.cc > b/osaf/services/saf/amf/amfd/hlttype.cc > --- a/osaf/services/saf/amf/amfd/hlttype.cc > +++ b/osaf/services/saf/amf/amfd/hlttype.cc > @@ -33,7 +33,8 @@ static SaAisErrorT hct_ccb_completed_cb( > rc = SA_AIS_OK; > break; > case CCBUTIL_MODIFY: > - report_ccb_validation_error(opdata, "Modification of > SaAmfHealthcheckType not supported"); > + // modifications are OK, node director re-reads type every now > and then > + rc = SA_AIS_OK; > break; > case CCBUTIL_DELETE: > rc = SA_AIS_OK; > @@ -55,6 +56,9 @@ static void hct_ccb_apply_cb(CcbUtilOper > break; > case CCBUTIL_DELETE: > break; > + case CCBUTIL_MODIFY: > + // values not used, no need to reinitialize type > + break; > default: > osafassert(0); > break; ------------------------------------------------------------------------------ Managing the Performance of Cloud-Based Applications Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. Read the Whitepaper. http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
