diff options
author | Ryan Rossiter <rlrossit@us.ibm.com> | 2015-08-11 15:10:48 +0000 |
---|---|---|
committer | Ryan Rossiter <rlrossit@us.ibm.com> | 2015-08-11 16:09:48 +0000 |
commit | 97821a9c7e1aff420ef2de319e158630ce811551 (patch) | |
tree | b7a5ab07894c1e0952d1eb9b00fc860548e18c0e /HACKING.rst | |
parent | 26fcd5d7c44850b35f4ca6606ad32fa5c6c1b87d (diff) | |
download | nova-97821a9c7e1aff420ef2de319e158630ce811551.tar.gz |
Add hacking check for greenthread.spawn()
Because greenthread.spawn() and spawn_n() are missing a nova context
(see I3623e60c49e442e2431cf017540422aa59bc285a and
Ia5175c9729069df3d779237ba6039cf5bc920018), nova.utils.spawn() and
spawn_n() should be used when spawning threads. This change adds a
hacking check to assert this is being done during pep8.
Change-Id: Id52c30bb5ded2184d772e6026b0f04f9a0efeb55
Related-Bug: #1404268
Closes-Bug: #1468513
Diffstat (limited to 'HACKING.rst')
-rw-r--r-- | HACKING.rst | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/HACKING.rst b/HACKING.rst index 1076469ddc..8683b2ed92 100644 --- a/HACKING.rst +++ b/HACKING.rst @@ -50,6 +50,7 @@ Nova Specific Commandments assertEqual(A in B, False) or assertEqual(False, A in B) to the more specific assertIn/NotIn(A, B) - [N339] Check common raise_feature_not_supported() is used for v2.1 HTTPNotImplemented response. +- [N340] Check nova.utils.spawn() is used instead of greenthread.spawn() Creating Unit Tests ------------------- |