diff options
author | Amar Tumballi <amarts@redhat.com> | 2018-07-27 23:11:51 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-09-04 09:15:49 +0000 |
commit | a105b25915a2b9c900957c71443e0be2ef5fa9ec (patch) | |
tree | 78861ad2ca826c2bef34c1e5e875d8ae68da52e5 /libglusterfs/src/options.h | |
parent | 53e6e621408bca1e053caa2f93af84788edb2dfa (diff) |
classification: provide infra to start labelling features/components
`doc/xlator-classification.md` talks about the reasoning and expectations
Reviewers are expected to check the 'category' of new
option / translator added in the codebase, and make sure the flag
is always properly set. It helps to keep the 'expectation' proper
on the codebase.
updates: #430
Change-Id: I2bfc9934a5f6eed77fcc3e20364046242decc82c
Signed-off-by: Amar Tumballi <amarts@redhat.com>
Diffstat (limited to 'libglusterfs/src/options.h')
-rw-r--r-- | libglusterfs/src/options.h | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/libglusterfs/src/options.h b/libglusterfs/src/options.h index 1aa27341624..1ce8975e799 100644 --- a/libglusterfs/src/options.h +++ b/libglusterfs/src/options.h @@ -134,9 +134,15 @@ typedef struct volume_options { */ char *setkey; - /* The level at which the option is classified - */ - opt_level_t level; + /* A 'level' is about the technical depth / understanding one + needs to handle the option. 'category' is based on + quality (ie, tests, people behind it, documentation available) */ + + /* The level at which the option is classified */ + opt_level_t level; + + /* Flag to understand how this option is categorized */ + gf_category_t category; } volume_option_t; |