Anders, What's your thoughts on moving this to nid.conf. CLM may be the implementer, but the implication is system wide which is why this should go into nid.conf that contains the common configuration.
Mathi. ----- mathi.naic...@oracle.com wrote: > Ack, > This variable should go into the 00-README.conf (and CLM PR too?) > And we should consider moving this variable to nid.conf and perhaps > call it > OSAF_ACTIVE_ELECTION_DELAY! > > Mathi. > > > > -----Original Message----- > > From: Anders Widell [mailto:anders.wid...@ericsson.com] > > Sent: Thursday, April 07, 2016 4:15 PM > > To: Mathivanan Naickan Palanivelu > > Cc: opensaf-devel@lists.sourceforge.net > > Subject: [PATCH 1 of 1] clm: Decrease the default election delay > time to 200 > > ms [#1724] > > > > osaf/services/saf/clmsv/config/clmna.conf | 5 +++-- > > osaf/services/saf/clmsv/nodeagent/election_starter.h | 2 +- > > 2 files changed, 4 insertions(+), 3 deletions(-) > > > > > > For backwards compatibility reasons, decrease the default election > delay > > time to > > 200 ms. Previously, the default election delay time was set to 5 > seconds, > > which is the recommended setting for clusters with more than two > system > > controllers. > > For clusters with only two system controllers, the new default of > 200 ms > > ought to be sufficient. > > > > diff --git a/osaf/services/saf/clmsv/config/clmna.conf > > b/osaf/services/saf/clmsv/config/clmna.conf > > --- a/osaf/services/saf/clmsv/config/clmna.conf > > +++ b/osaf/services/saf/clmsv/config/clmna.conf > > @@ -14,8 +14,9 @@ export CLMNA_ENV_HEALTHCHECK_KEY="Defaul # > > STANDBY). If no system controller has been seen during the > configured time > > # period, CLMNA will initate an election with the own node as a > candidate for > > a # new active system controller. Note that lowering this value > will increase > > the -# risk of split-brain. Values lower than 5000 are not > recommended. > > -#export CLMNA_ELECTION_DELAY_TIME=5000 > > +# risk of split-brain. Values lower than 5000 are not recommended > on > > +cluster # with more than two system controllers. > > +#export CLMNA_ELECTION_DELAY_TIME=200 > > > > # Uncomment the next line to enable info level logging #args="-- > > loglevel=info" > > diff --git a/osaf/services/saf/clmsv/nodeagent/election_starter.h > > b/osaf/services/saf/clmsv/nodeagent/election_starter.h > > --- a/osaf/services/saf/clmsv/nodeagent/election_starter.h > > +++ b/osaf/services/saf/clmsv/nodeagent/election_starter.h > > @@ -66,7 +66,7 @@ class ElectionStarter { > > timespec Poll(); > > > > private: > > - static const uint64_t kDefaultElectionDelayTime = 5000; > > + static const uint64_t kDefaultElectionDelayTime = 200; > > > > struct NodeCollection { > > NodeCollection(); > > ------------------------------------------------------------------------------ > _______________________________________________ > Opensaf-devel mailing list > Opensaf-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/opensaf-devel ------------------------------------------------------------------------------ _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel
