Merge "Add docs section on config options to reviewers guide"
This commit is contained in:
commit
7c7280edd8
@ -51,6 +51,19 @@ skipped or not. Do not approve changes that depend on an API call to determine
|
|||||||
whether to skip or not.
|
whether to skip or not.
|
||||||
|
|
||||||
|
|
||||||
|
Configuration Options
|
||||||
|
---------------------
|
||||||
|
With the introduction of the tempest external test plugin interface we needed
|
||||||
|
to provide a stable contract for tempest's configuration options. This means
|
||||||
|
we can no longer simply remove a configuration option when it's no longer used.
|
||||||
|
Patches proposed that remove options without a deprecation cycle should not
|
||||||
|
be approved. Similarly when changing default values with configuration we need
|
||||||
|
to similarly be careful that we don't break existing functionality. Also, when
|
||||||
|
adding options, just as before, we need to weigh the benefit of adding an
|
||||||
|
additional option against the complexity and maintenance overhead having it
|
||||||
|
costs.
|
||||||
|
|
||||||
|
|
||||||
Test Documentation
|
Test Documentation
|
||||||
------------------
|
------------------
|
||||||
When a new test is being added refer to the :ref:`TestDocumentation` section in
|
When a new test is being added refer to the :ref:`TestDocumentation` section in
|
||||||
|
Loading…
Reference in New Issue
Block a user