diff options
author | raghavendra talur <raghavendra.talur@gmail.com> | 2015-08-20 15:09:31 +0530 |
---|---|---|
committer | Humble Devassy Chirammal <humble.devassy@gmail.com> | 2015-08-31 02:27:22 -0700 |
commit | 9e9e3c5620882d2f769694996ff4d7e0cf36cc2b (patch) | |
tree | 3a00cbd0cc24eb7df3de9b2eeeb8d42ee9175f88 /README.md | |
parent | f6055cdb4dedde576ed8ec55a13814a69dceefdc (diff) |
Create basic directory structure
All new features specs go into in_progress directory.
Once signed off, it should be moved to done directory.
For now,
This change moves all the Gluster 4.0 feature specs to
in_progress. All other specs are under done/release-version.
More cleanup required will be done incrementally.
Change-Id: Id272d301ba8c434cbf7a9a966ceba05fe63b230d
BUG: 1206539
Signed-off-by: Raghavendra Talur <rtalur@redhat.com>
Reviewed-on: http://review.gluster.org/11969
Reviewed-by: Humble Devassy Chirammal <humble.devassy@gmail.com>
Reviewed-by: Prashanth Pai <ppai@redhat.com>
Tested-by: Humble Devassy Chirammal <humble.devassy@gmail.com>
Diffstat (limited to 'README.md')
-rw-r--r-- | README.md | 57 |
1 files changed, 53 insertions, 4 deletions
@@ -1,9 +1,58 @@ -# Contributing +Gluster Specs Repository +------------------------ -Contributing to the glusterfs-specs repository follows the same steps for glusterfs, with one small difference. +This is a git repository for doing design review on enhancements to +Gluster. This provides an ability to ensure that everyone +has signed off on the approach to solving a problem early on. -You run `git review` instead of `./rfc.sh`. +Repository Structure +-------------------- -The `git-review` tool can be installed from the package repositories for your distribution. +The structure of the repository is as follows:: +``` +. ++-- done/ +| +-- release-3.4/ +| +-- release-3.5/ +| +-- release-3.6/ +| +-- release-3.7/ ++-- in_progress/ +``` + +Implemented specs will be moved to ``done`` once the associated code has landed. + +The Flow of an Idea from your Head to Implementation +---------------------------------------------------- +First propose a spec to the ``in_progress`` directory so that it can be reviewed. Reviewers adding a positive +1/+2 review in gerrit are promising that they will review the code when it is proposed. Spec documents should be approved and merged as soon as possible, and spec documents in the ``in_progress`` directory can be updated as often as needed. Iterate on it. + +1. Have an idea +2. Propose a spec. +3. Reviewers review the spec. As soon as 2 core reviewers like something, merge it. Iterate on the spec as often as needed, and keep it updated. +4. Once there is agreement on the spec, write the code. +5. As the code changes during review, keep the spec updated as needed. +6. Once the code lands (with all necessary tests and docs), the spec can be moved to the ``done`` directory. If a feature needs a spec, it needs docs, and the docs must land before or with the feature (not after). + +Spec Lifecycle Rules +-------------------- +1. Land quickly: A spec is a living document, and lives in the repository not in gerrit. +2. Initial version is an idea not a technical design: That way the merits of the idea can be discussed and landed and not stuck in gerrit limbo land. +3. Second version is an overview of the technical design: This will aid in the technical discussions amongst the community. +4. Subsequent versions improve/enhance technical design: Each of these versions should be relatively small patches to the spec to keep rule #1. And keeps the spec up to date with the progress of the implementation. + +How to submit a new feature for review +-------------------------------------- +1. Clone this repo +2. Copy the ``template.md`` file in ``in_progress`` directory to ``your_feature.md`` +3. Make changes to ``your_feature.md`` +4. Submit changes using ``git-review`` tool. + + +How to ask questions and get clarifications about a spec +-------------------------------------------------------- +To make a comment, suggestion or to ask a question use the gerrit interface like you do for patches on glusterfs project. + +Learn As We Go +-------------- +This version of README is largely inspired from openstack-swift project. We can change the process and update this README from our learning and adapt. |