All,
Thanks to Nigel, this is now deployed, and any new patches referencing github (ie, new features) need the 'DocApproved' and 'SpecApproved' label.
Regards,
Amar
On Mon, Apr 2, 2018 at 10:40 AM, Amar Tumballi <atumball@xxxxxxxxxx> wrote:
Hi all,A better documentation about the feature, and also information about how to use the features are one of the major ask of the community when they want to use glusterfs, or want to contribute by helping get the features, bug fixes for features, etc.Finally, we have taken some baby steps to get that ask of having better design and documentation resolved. We had discussed this in our automation goals [1], to make having design spec, and documentation mandatory for a feature patch. Now, thanks to Shyam and Nigel, we have the patch ready to automate this process [2].Feel free to review the patch, and comment on this.A heads up on how it looks like after this patch gets in.* A patch for a github reference won't pass smoke unless these labels are present on github issue.* Everyone, feel free to review and comment on the issue / patch regarding the document. But, the label is expected to be provided only by Project's general architects, and any industry experts we as community nominate for validating feature. Initially for making sure we have a valid process, where I don't provides flags quickly, expectation is to have two people comment about approving the flags, and then the label can be provided.* Some may argue, the rate of development can reduce if we make this flag mandatory, but what is the use of having a feature without design and documentation on how to use it?For those who want to provide Spec and Doc approved flags, you can have a quick link [3], to see all the tests which fail smoke. Not all smoke failures would be for missing Spec and Doc flag, but this is just a quick start.We would like to implement this check soon, and happy to accommodate the feedback and suggestions along the way.Regards,Amar
Amar Tumballi (amarts)
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://lists.gluster.org/mailman/listinfo/gluster-devel
