Hi! Comments for Chapter 6:
1) Name of the chapter. Although we actually have implemented support for limited functionality during the time when both system controllers are absent, this is not the main purpose of the feature and thus I think the name of the chapter is misleading. The main purpose of the sc absence feature is to let the cluster survive a (near) simultaneous failure of both system controllers. Maybe it would be better to simply give the chapter the same name as the feature: "SC Absence", or "System Controller Absence". 2) OpenSAF version: Didn't we introduce this feature already in OpenSAF 5.0, not 5.1 as stated here? 3) List of services that support this feature: Although SMF doesn't provide any service while the system controllers are absent, the SMF service does support and is compatible with the SC absence feature. I think we should list the OpenSAF services that are compatible with (and tested with) the feature, regardless of whether they provide any service or not during a period of SC absence. 4) Section 6.1 - 6.6 should be deleted. We shouldn't duplicate the text from other PRs in here, and I don't think it is a good idea to provide a reference to the exact section numbers in other documents either - these documents may later be modified so that the reference is no longer pointing to the correct section, and there would be a high risk that we forget to update the references in this document. We could instead simply with one sentence mention that the PRs for each individual service gives additional details of how SC absence is handled by that OpenSAF service, without pointing to any exact section in any of these documents. regards, Anders Widell On 09/23/2016 04:15 PM, Mathivanan Naickan Palanivelu wrote: > Hi Anders, > > I have created a section 6 primarily to give an outline and structure to the > topic of 'sc absence'. > Have provided pointers to individual service specific PRs. This could be the > first stab. Let me know your thoughts. > Will also change the name of the READMEs of LOG and CKPT to have the suffix > as SC_ABSENCE (for consistency sakes). > > Thanks, > Mathi. ------------------------------------------------------------------------------ _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
