Fix spec template formatting
No change to any verbiage here. Just comply with the following formatting requirement in the template: * Please wrap text at 79 columns. Change-Id: I02871d2d999ccd6063720e66b6de0cb3b666018d Signed-off-by: Robert Church <robert.church@windriver.com>
This commit is contained in:
parent
f7c04656d8
commit
08ff127d4b
|
@ -1,6 +1,10 @@
|
||||||
.. This work is licensed under a Creative Commons Attribution 3.0 Unported License. http://creativecommons.org/licenses/by/3.0/legalcode
|
..
|
||||||
|
This work is licensed under a Creative Commons Attribution 3.0 Unported
|
||||||
|
License. http://creativecommons.org/licenses/by/3.0/legalcode
|
||||||
|
|
||||||
.. Many thanks to the OpenStack Nova team for the Example Spec that formed the basis for this document.
|
..
|
||||||
|
Many thanks to the OpenStack Nova team for the Example Spec that formed the
|
||||||
|
basis for this document.
|
||||||
|
|
||||||
=======================
|
=======================
|
||||||
StarlingX: Example Spec
|
StarlingX: Example Spec
|
||||||
|
@ -8,25 +12,23 @@ StarlingX: Example Spec
|
||||||
|
|
||||||
Some notes about the Spec process:
|
Some notes about the Spec process:
|
||||||
|
|
||||||
* The aim of this document is first to define the problem we need to solve,
|
* The aim of this document is first to define the problem we need to solve, and
|
||||||
and second agree on the overall approach to solve that problem.
|
second agree on the overall approach to solve that problem.
|
||||||
|
|
||||||
* This is not intended to be extensive documentation for a new feature.
|
* This is not intended to be extensive documentation for a new feature. For
|
||||||
For example, there is no need to specify the exact configuration changes,
|
example, there is no need to specify the exact configuration changes, nor the
|
||||||
nor the exact details of any DB model changes. But you should still define
|
exact details of any DB model changes. But you should still define that such
|
||||||
that such changes are required, and be clear on how that will affect
|
changes are required, and be clear on how that will affect upgrades.
|
||||||
upgrades.
|
|
||||||
|
|
||||||
* You should aim to get your spec approved before writing your code.
|
* You should aim to get your spec approved before writing your code. While you
|
||||||
While you are free to write prototypes and code before getting your spec
|
are free to write prototypes and code before getting your spec approved, its
|
||||||
approved, its possible that the outcome of the spec review process leads
|
possible that the outcome of the spec review process leads you towards a
|
||||||
you towards a fundamentally different solution than you first envisaged.
|
fundamentally different solution than you first envisaged.
|
||||||
|
|
||||||
* But, API changes are held to a much higher level of scrutiny.
|
* But, API changes are held to a much higher level of scrutiny. As soon as an
|
||||||
As soon as an API change merges, we must assume it could be in production
|
API change merges, we must assume it could be in production somewhere, and as
|
||||||
somewhere, and as such, we then need to support that API change forever.
|
such, we then need to support that API change forever. To avoid getting that
|
||||||
To avoid getting that wrong, we do want lots of details about API changes
|
wrong, we do want lots of details about API changes upfront.
|
||||||
upfront.
|
|
||||||
|
|
||||||
Some notes about using this template:
|
Some notes about using this template:
|
||||||
|
|
||||||
|
@ -34,12 +36,13 @@ Some notes about using this template:
|
||||||
|
|
||||||
* Please wrap text at 79 columns.
|
* Please wrap text at 79 columns.
|
||||||
|
|
||||||
* The filename in the git repository should include the StoryBoard number and name,
|
* The filename in the git repository should include the StoryBoard number and
|
||||||
for example a Story at https://storyboard.openstack.org/#!/story/1234567
|
name, for example a Story at
|
||||||
should be named [category]_1234567-feature-name.rst (refer to instructions.rst
|
https://storyboard.openstack.org/#!/story/1234567 should be named
|
||||||
for guidelines on a suitable category name)
|
[category]_1234567-feature-name.rst (refer to instructions.rst for guidelines
|
||||||
|
on a suitable category name)
|
||||||
|
|
||||||
* Please do not delete any of the sections in this template. If you have
|
* Please do not delete any of the sections in this template. If you have
|
||||||
nothing to say for a whole section, just write: None
|
nothing to say for a whole section, just write: None
|
||||||
|
|
||||||
* For help with syntax, see http://sphinx-doc.org/rest.html
|
* For help with syntax, see http://sphinx-doc.org/rest.html
|
||||||
|
@ -48,11 +51,11 @@ Some notes about using this template:
|
||||||
HTML file in doc/build/html/specs/<path_of_your_file>
|
HTML file in doc/build/html/specs/<path_of_your_file>
|
||||||
|
|
||||||
* If you would like to provide a diagram with your spec, ascii diagrams are
|
* If you would like to provide a diagram with your spec, ascii diagrams are
|
||||||
required. http://asciiflow.com/ is a very nice tool to assist with making
|
required. http://asciiflow.com/ is a very nice tool to assist with making
|
||||||
ascii diagrams. The reason for this is that the tool used to review specs is
|
ascii diagrams. The reason for this is that the tool used to review specs is
|
||||||
based purely on plain text. Plain text will allow review to proceed without
|
based purely on plain text. Plain text will allow review to proceed without
|
||||||
having to look at additional files which can not be viewed in gerrit. It
|
having to look at additional files which can not be viewed in gerrit. It will
|
||||||
will also allow inline feedback on the diagram itself.
|
also allow inline feedback on the diagram itself.
|
||||||
|
|
||||||
|
|
||||||
Example Spec - The title of your blueprint
|
Example Spec - The title of your blueprint
|
||||||
|
@ -60,24 +63,26 @@ Example Spec - The title of your blueprint
|
||||||
|
|
||||||
Include the URL of your Storyboard Story:
|
Include the URL of your Storyboard Story:
|
||||||
|
|
||||||
Storyboard: https://storyboard.openstack.org/#!/story/list?status=active&project_group_id=86
|
Storyboard:
|
||||||
|
https://storyboard.openstack.org/#!/story/list?status=active&project_group_id=86
|
||||||
|
|
||||||
Introduction paragraph -- why are we doing anything? The essential "Why" or motivation is key to laying the ground for the work ahead. It provides contexts for all involved in the work. A single paragraph of
|
Introduction paragraph -- why are we doing anything? The essential "Why" or
|
||||||
prose that operators can understand. The title and this first paragraph
|
motivation is key to laying the ground for the work ahead. It provides contexts
|
||||||
should be used as the subject line and body of the commit message
|
for all involved in the work. A single paragraph of prose that operators can
|
||||||
respectively.
|
understand. The title and this first paragraph should be used as the subject
|
||||||
|
line and body of the commit message respectively.
|
||||||
|
|
||||||
Problem description
|
Problem description
|
||||||
===================
|
===================
|
||||||
|
|
||||||
A detailed description of the problem. What problem is this spec
|
A detailed description of the problem. What problem is this spec addressing?
|
||||||
addressing?
|
|
||||||
|
|
||||||
Use Cases
|
Use Cases
|
||||||
=========
|
=========
|
||||||
|
|
||||||
What use cases does this address? What impact on actors does this change have?
|
What use cases does this address? What impact on actors does this change have?
|
||||||
Ensure you are clear about the actors/personas in each use case: Developer, End User, Deployer etc.
|
Ensure you are clear about the actors/personas in each use case: Developer, End
|
||||||
|
User, Deployer etc.
|
||||||
|
|
||||||
Proposed change
|
Proposed change
|
||||||
===============
|
===============
|
||||||
|
@ -89,24 +94,25 @@ If this is one part of a larger effort make it clear where this piece ends. In
|
||||||
other words, what's the scope of this effort?
|
other words, what's the scope of this effort?
|
||||||
|
|
||||||
At this point, if you would like to just get feedback on if the problem and
|
At this point, if you would like to just get feedback on if the problem and
|
||||||
proposed change fit in StarlingX, you can stop here and post this for review to get preliminary feedback. If so please say:
|
proposed change fit in StarlingX, you can stop here and post this for review to
|
||||||
Posting to get preliminary feedback on the scope of this spec.
|
get preliminary feedback. If so please say: Posting to get preliminary feedback
|
||||||
|
on the scope of this spec.
|
||||||
|
|
||||||
Alternatives
|
Alternatives
|
||||||
============
|
============
|
||||||
|
|
||||||
What other ways could we do this thing? Why aren't we using those? This
|
What other ways could we do this thing? Why aren't we using those? This doesn't
|
||||||
doesn't have to be a full literature review, but it should demonstrate that
|
have to be a full literature review, but it should demonstrate that thought has
|
||||||
thought has been put into why the proposed solution is an appropriate one.
|
been put into why the proposed solution is an appropriate one.
|
||||||
|
|
||||||
Data model impact
|
Data model impact
|
||||||
=================
|
=================
|
||||||
|
|
||||||
Changes which require modifications to the data model often have a wider
|
Changes which require modifications to the data model often have a wider impact
|
||||||
impact on the system. The community often has strong opinions on how the data
|
on the system. The community often has strong opinions on how the data model
|
||||||
model should be evolved, from both a functional and performance perspective.
|
should be evolved, from both a functional and performance perspective. It is
|
||||||
It is therefore important to capture and gain agreement as early as possible
|
therefore important to capture and gain agreement as early as possible on any
|
||||||
on any proposed changes to the data model.
|
proposed changes to the data model.
|
||||||
|
|
||||||
Questions which need to be addressed by this section should include:
|
Questions which need to be addressed by this section should include:
|
||||||
|
|
||||||
|
@ -122,54 +128,53 @@ REST API impact
|
||||||
|
|
||||||
Each API method which is either added or changed should have the following
|
Each API method which is either added or changed should have the following
|
||||||
|
|
||||||
* Specification for the method : As best as can be determined at
|
* Specification for the method : As best as can be determined at the definition
|
||||||
the definition stage.
|
stage.
|
||||||
|
|
||||||
* Parameters which can be passed via the url
|
* Parameters which can be passed via the url
|
||||||
|
|
||||||
* Example use case including typical API samples for both data supplied
|
* Example use case including typical API samples for both data supplied by the
|
||||||
by the caller and the response
|
caller and the response
|
||||||
|
|
||||||
* Discuss any policy changes, and discuss what things a deployer needs to
|
* Discuss any policy changes, and discuss what things a deployer needs to think
|
||||||
think about when defining their policy.
|
about when defining their policy.
|
||||||
|
|
||||||
Note that the schema should be defined as restrictively as
|
Note that the schema should be defined as restrictively as possible. Parameters
|
||||||
possible. Parameters which are required should be marked as such and
|
which are required should be marked as such and only under exceptional
|
||||||
only under exceptional circumstances should additional parameters
|
circumstances should additional parameters which are not defined in the schema
|
||||||
which are not defined in the schema be permitted (eg
|
be permitted (eg additionaProperties should be False).
|
||||||
additionaProperties should be False).
|
|
||||||
|
|
||||||
Reuse of existing predefined parameter types such as regexps for
|
Reuse of existing predefined parameter types such as regexps for passwords and
|
||||||
passwords and user defined names is highly encouraged.
|
user defined names is highly encouraged.
|
||||||
|
|
||||||
Security impact
|
Security impact
|
||||||
===============
|
===============
|
||||||
|
|
||||||
Describe any potential security impact on the system. Some of the items to
|
Describe any potential security impact on the system. Some of the items to
|
||||||
consider include:
|
consider include:
|
||||||
|
|
||||||
* Does this change touch sensitive data such as tokens, keys, or user data?
|
* Does this change touch sensitive data such as tokens, keys, or user data?
|
||||||
|
|
||||||
* Does this change alter the API in a way that may impact security, such as
|
* Does this change alter the API in a way that may impact security, such as a
|
||||||
a new way to access sensitive information or a new way to login?
|
new way to access sensitive information or a new way to login?
|
||||||
|
|
||||||
* Does this change involve cryptography or hashing?
|
* Does this change involve cryptography or hashing?
|
||||||
|
|
||||||
* Does this change require the use of sudo or any elevated privileges?
|
* Does this change require the use of sudo or any elevated privileges?
|
||||||
|
|
||||||
* Does this change involve using or parsing user-provided data? This could
|
* Does this change involve using or parsing user-provided data? This could be
|
||||||
be directly at the API level or indirectly such as changes to a cache layer.
|
directly at the API level or indirectly such as changes to a cache layer.
|
||||||
|
|
||||||
* Can this change enable a resource exhaustion attack, such as allowing a
|
* Can this change enable a resource exhaustion attack, such as allowing a
|
||||||
single API interaction to consume significant server resources? Some examples
|
single API interaction to consume significant server resources? Some examples
|
||||||
of this include launching subprocesses for each connection, or entity
|
of this include launching subprocesses for each connection, or entity
|
||||||
expansion attacks in XML.
|
expansion attacks in XML.
|
||||||
|
|
||||||
For more detailed guidance, please see the OpenStack Security Guidelines as
|
For more detailed guidance, please see the OpenStack Security Guidelines as a
|
||||||
a reference (https://wiki.openstack.org/wiki/Security/Guidelines). These
|
reference (https://wiki.openstack.org/wiki/Security/Guidelines). These
|
||||||
guidelines are a work in progress and are designed to help you identify
|
guidelines are a work in progress and are designed to help you identify
|
||||||
security best practices. For further information, feel free to reach out
|
security best practices. For further information, feel free to reach out to the
|
||||||
to the OpenStack Security Group at openstack-security@lists.openstack.org.
|
OpenStack Security Group at openstack-security@lists.openstack.org.
|
||||||
|
|
||||||
Other end user impact
|
Other end user impact
|
||||||
=====================
|
=====================
|
||||||
|
@ -183,9 +188,9 @@ feature?
|
||||||
Performance Impact
|
Performance Impact
|
||||||
==================
|
==================
|
||||||
|
|
||||||
Describe any potential performance impact on the system, for example
|
Describe any potential performance impact on the system, for example how often
|
||||||
how often will new code be called, and is there a major change to the calling
|
will new code be called, and is there a major change to the calling pattern of
|
||||||
pattern of existing code.
|
existing code.
|
||||||
|
|
||||||
Examples of things to consider here include:
|
Examples of things to consider here include:
|
||||||
|
|
||||||
|
@ -207,12 +212,12 @@ Examples of things to consider here include:
|
||||||
Other deployer impact
|
Other deployer impact
|
||||||
=====================
|
=====================
|
||||||
|
|
||||||
Discuss things that will affect how you deploy and configure OpenStack
|
Discuss things that will affect how you deploy and configure OpenStack that
|
||||||
that have not already been mentioned, such as:
|
have not already been mentioned, such as:
|
||||||
|
|
||||||
* What config options are being added? Should they be more generic than
|
* What config options are being added? Should they be more generic than
|
||||||
proposed? Are the default values ones which will work well in
|
proposed? Are the default values ones which will work well in real
|
||||||
real deployments?
|
deployments?
|
||||||
|
|
||||||
* Is this a change that takes immediate effect after its merged, or is it
|
* Is this a change that takes immediate effect after its merged, or is it
|
||||||
something that has to be explicitly enabled?
|
something that has to be explicitly enabled?
|
||||||
|
@ -221,7 +226,7 @@ that have not already been mentioned, such as:
|
||||||
|
|
||||||
* Please state anything that those those upgrading from the previous release,
|
* Please state anything that those those upgrading from the previous release,
|
||||||
need to be aware of. Also describe any plans to deprecate configuration
|
need to be aware of. Also describe any plans to deprecate configuration
|
||||||
values or features. Consider the potential implications of automated
|
values or features. Consider the potential implications of automated
|
||||||
deployment technologies.
|
deployment technologies.
|
||||||
|
|
||||||
Developer impact
|
Developer impact
|
||||||
|
@ -234,12 +239,12 @@ Upgrade impact
|
||||||
|
|
||||||
Describe any potential upgrade impact on the system, such as:
|
Describe any potential upgrade impact on the system, such as:
|
||||||
|
|
||||||
* StarlingX supports N-1 version for rolling upgrades. Does
|
* StarlingX supports N-1 version for rolling upgrades. Does the proposed change
|
||||||
the proposed change need to consider older code running that may impact how
|
need to consider older code running that may impact how the new change
|
||||||
the new change functions, for example, by changing or overwriting global
|
functions, for example, by changing or overwriting global state in the
|
||||||
state in the database? This is generally most problematic when making changes
|
database? This is generally most problematic when making changes that involve
|
||||||
that involve multiple compute hosts, like move operations such as migrate,
|
multiple compute hosts, like move operations such as migrate, resize,
|
||||||
resize, unshelve and evacuate.
|
unshelve and evacuate.
|
||||||
|
|
||||||
|
|
||||||
Implementation
|
Implementation
|
||||||
|
@ -276,8 +281,8 @@ but we're mostly trying to understand the timeline for implementation.
|
||||||
Dependencies
|
Dependencies
|
||||||
============
|
============
|
||||||
|
|
||||||
* Include specific references to specs in StarlingX, or in other
|
* Include specific references to specs in StarlingX, or in other projects, that
|
||||||
projects, that this one either depends on or is related to.
|
this one either depends on or is related to.
|
||||||
|
|
||||||
* If this requires functionality of another project that is not currently used
|
* If this requires functionality of another project that is not currently used
|
||||||
by StarlingX document that fact.
|
by StarlingX document that fact.
|
||||||
|
@ -289,16 +294,16 @@ Dependencies
|
||||||
Testing
|
Testing
|
||||||
=======
|
=======
|
||||||
|
|
||||||
Please discuss the important scenarios needed to test here, as well as
|
Please discuss the important scenarios needed to test here, as well as specific
|
||||||
specific edge cases we should be ensuring work correctly. For each
|
edge cases we should be ensuring work correctly. For each scenario please
|
||||||
scenario please specify if this requires specialized hardware, a full
|
specify if this requires specialized hardware, a full openstack environment, or
|
||||||
openstack environment, or can be simulated inside the project tree.
|
can be simulated inside the project tree.
|
||||||
|
|
||||||
Please discuss how the change will be tested. We especially want to know what
|
Please discuss how the change will be tested. We especially want to know what
|
||||||
tempest tests will be added. It is assumed that unit test coverage will be
|
tempest tests will be added. It is assumed that unit test coverage will be
|
||||||
added so that doesn't need to be mentioned explicitly, but discussion of why
|
added so that doesn't need to be mentioned explicitly, but discussion of why
|
||||||
you think unit tests are sufficient and we don't need to add more
|
you think unit tests are sufficient and we don't need to add more tests would
|
||||||
tests would need to be included.
|
need to be included.
|
||||||
|
|
||||||
Is this untestable in gate given current limitations (specific hardware /
|
Is this untestable in gate given current limitations (specific hardware /
|
||||||
software configurations available)? If so, are there mitigation plans (3rd
|
software configurations available)? If so, are there mitigation plans (3rd
|
||||||
|
@ -309,12 +314,12 @@ Documentation Impact
|
||||||
====================
|
====================
|
||||||
|
|
||||||
Which audiences are affected most by this change, and which documentation
|
Which audiences are affected most by this change, and which documentation
|
||||||
titles for StarlingX should be updated because of this change? Don't
|
titles for StarlingX should be updated because of this change? Don't repeat
|
||||||
repeat details discussed above, but reference them here in the context of
|
details discussed above, but reference them here in the context of
|
||||||
documentation for multiple audiences. For example, the End User Guide would
|
documentation for multiple audiences. For example, the End User Guide would
|
||||||
need to be updated if the change offers a new feature available through the
|
need to be updated if the change offers a new feature available through the CLI
|
||||||
CLI or dashboard. If a config option changes or is deprecated, note here that
|
or dashboard. If a config option changes or is deprecated, note here that the
|
||||||
the documentation needs to be updated to reflect this specification's change.
|
documentation needs to be updated to reflect this specification's change.
|
||||||
|
|
||||||
References
|
References
|
||||||
==========
|
==========
|
||||||
|
@ -329,7 +334,7 @@ references are unavailable. Examples of what you could include are:
|
||||||
|
|
||||||
* Links to relevant research, if appropriate
|
* Links to relevant research, if appropriate
|
||||||
|
|
||||||
* Related specifications as appropriate (e.g. if it's an EC2 thing, link the
|
* Related specifications as appropriate (e.g. if it's an EC2 thing, link the
|
||||||
EC2 docs)
|
EC2 docs)
|
||||||
|
|
||||||
* Anything else you feel it is worthwhile to refer to
|
* Anything else you feel it is worthwhile to refer to
|
||||||
|
@ -349,4 +354,3 @@ what's happened along the time.
|
||||||
- Description
|
- Description
|
||||||
* - Stein
|
* - Stein
|
||||||
- Introduced
|
- Introduced
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue