summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | | | | | | | | Merge "Add host name to log message for _local_delete"Jenkins2013-01-111-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | Add host name to log message for _local_deleteJian Wen2013-01-101-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Better log message will hopefully make it easier for operators to diagnose. Change-Id: I447f7530a05f9af6e152f1e5d484027684795b27
* | | | | | | | | | | | | | | | | | | | Merge "Create a directory for servicegroup drivers."Jenkins2013-01-113-1/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | Create a directory for servicegroup drivers.Russell Bryant2013-01-103-1/+3
| | |_|_|_|/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now there is only one servicegroup driver, the db backed driver. Create a directory for drivers to reside in to keep things a bit more tidy as we add additional drivers. Part of blueprint rpc-based-servicegroup-driver. Change-Id: Ib563e1a8d184cef838e5730b2fc6904940d04c21
* | | | | | | | | | | | | | | | | | | Merge "Move logic from os-api-host into compute"Jenkins2013-01-119-236/+505
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | Move logic from os-api-host into computeMatthew Sherborne2013-01-119-236/+505
| | |_|_|_|_|_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Renames host to host_name in various places to make it explicit what it is the APIs are accepting and what we are passing around. Moves logic out of the openstack api's host extension to the compute library. This is in preparation for cells; cells provides its own compute library that proxies certain calls to child cells. In an effort to separate out the layers of code, we're moving all DB related logic out of the openstack api layer and into compute library layer. For example @check_host which calls the DB should not be in the openstack api layer. In the case of cells, we can't check for the existence of the host until we arrive at the cell that contains the host. Implements: blueprint host-api-prep-for-cells Change-Id: Icb861fd323ba707ab3b661f0a2c6d8bb7bfdc85e
* | | | | | | | | | | | | | | | | | | Merge "Access instance as dict, not object in xenapi"Jenkins2013-01-112-16/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Access instance as dict, not object in xenapiChris Behrens2013-01-112-16/+6
| |/ / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_all_bw_counters() in XenAPI was accessing instances as objects instead of dicts. Fixes bug 1098382 Change-Id: I9a783c0f403b60d392dc26a25b93683e7c145600
* | | | | | | | | | | | | | | | | | Merge "Move update_instance_info_cache to conductor."Jenkins2013-01-118-35/+101
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Move update_instance_info_cache to conductor.Chris Behrens2013-01-108-35/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The network API always updates the instance_info_caches table whenever someone asks for current network information for an instance. When the compute manager calls network_api.get_instance_nw_info(), we'll need to use the conductor for the cache update instead of a local DB call. Adds an option to network_api's get_instance_nw_info() to not update the cache and adds code into compute manager to update the cache via the conductor. Related to blueprint no-db-compute-manager Change-Id: I36c3eb5b4e31ef2e26d3023bf2521d95205eb6ac
* | | | | | | | | | | | | | | | | | | Change ComputerDriver.legacy_nwinfo to raise by default.Robert Collins2013-01-116-5/+22
| |/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes non-updated hypervisors visible by grepping for 'def legacy_nwinfo' rather than only the updated ones being visible, and when new hypervisors are added, it will be clear whether they use the legacy format or not. Out of tree hypervisors will be broken by this, but the fix is trivial:: def legacy_nwinfo(self): # XXX TODO Update to use the non-legacy format. return True Change-Id: If5b461bc5d8e8dc21de3ca9cf521e7b341724900
* | | | | | | | | | | | | | | | | | Merge "Update instance's cell_name in API cell."Jenkins2013-01-102-6/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Update instance's cell_name in API cell.Chris Behrens2013-01-102-6/+7
| | |_|_|_|/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was accidentally being removed before updating the DB due to code being moved and not updated. The test for the method was broken, too, allowing this bug to sneak in. Fixes bug 1098093 Change-Id: I05c00b8b100568b39afb8ee0ca7ee94d4dd97c0a
* | | | | | | | | | | | | | | | | | Merge "Add user/tenant shim to RequestContext"Jenkins2013-01-101-1/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Add user/tenant shim to RequestContextRick Harris2013-01-101-1/+16
| | |/ / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is necessary because we pass the Nova context object to openstack-common/code (logging, for example), which expects the attributes to be named user/tenant not user_id/project_id. Fixes bug 1098278 Change-Id: Ic0dc49ef54515fc0ffaa0895cf4d88701afb1e16
* | | | | | | | | | | | | | | | | Merge "make runtests -p act more like tox"Jenkins2013-01-102-6/+12
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | make runtests -p act more like toxSean Dague2013-01-102-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | now that all the N402 fixes are in, start enforcing it Change-Id: I9d495849bf698e5af142d0ccafe551ac0e8a600c
* | | | | | | | | | | | | | | | | | Merge "fix new N402 errors"Jenkins2013-01-1016-55/+53
|\| | | | | | | | | | | | | | | | | | |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | fix new N402 errorsSean Dague2013-01-1016-55/+53
| |/ / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix the N402 errors that have slipped in in the last 48 hrs since starting this patch series. fix an N401 error that our scanner current can't find because it only looks for doc strings on classes and defs. this is the xeno's paradox of patch series, but we're getting close. Change-Id: I4a763bb4c812335d853eae05c72464f18ab93297
* | | | | | | | | | | | | | | | Merge "Fixed comment typo"Jenkins2013-01-101-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Fixed comment typoGiampaolo Lauria2013-01-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I68dd463c7365b536396c6715efa6042b3f102670
* | | | | | | | | | | | | | | | | Merge "Fix init_host checking moved instances"Jenkins2013-01-104-304/+167
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Fix init_host checking moved instancesChris Behrens2013-01-104-304/+167
| | |_|_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 6fc00d3465185 (https://review.openstack.org/#/c/18486/) introduced a couple of problems. 1) It introduced a method to try to deduce instance integer id from a name the driver reports and using instance_name_template. This fails badly when you're using something like 'instance-%(uuid)s' for instance_name_template. It's only used when the driver does not support the 'list_instance_uuids' method.. but XenAPI happens to not implement this method. 2) There's a call to _get_instance_volume_block_device_info() that is passing an instance_uuid when the method wants an instance. Fixes bug 1097987 (#1 above) by doing a brute force match against all instances' names. Fixes bug 1097978 (#2 above) Change-Id: I2da7658dd906fa06f0e867a2b22bf14e7f34cb12
* | | | | | | | | | | | | | | | Merge "Try out a new nova.conf.sample format"Jenkins2013-01-102-1359/+1766
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Try out a new nova.conf.sample formatMark McLoughlin2013-01-102-1359/+1766
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current format strikes me as fairly bizarre: - Having the help message after the option it's documenting seems backwards to me. Compare to the glance or swift samples. - Printing the type as e.g. "(StrOpt)" isn't very user friendly - We're creating a global pound symbol shortage Switch to a new format that looks much more like a normal config file. Change-Id: Ifba0a181667f3b81cad400f1cdd3812e388a9d64
* | | | | | | | | | | | | | | | Merge "Regenerate nova.conf.sample"Jenkins2013-01-102-440/+815
|\| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Regenerate nova.conf.sampleMark McLoughlin2013-01-102-440/+815
| | |_|/ / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some pretty monstrous hacking in the extraction tool is required to guess the group which options are in. It was pretty hacky to begin with though. Change-Id: Ie93dcfc82ab19af4e3cb7090bbb03d9efee83aa2
* | | | | | | | | | | | | | | Merge "Enhance wsgi to listen on ipv6 address"Jenkins2013-01-103-2/+26
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Enhance wsgi to listen on ipv6 addressDavanum Srinivas2013-01-093-2/+26
| | |_|_|_|_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Check if the hostname is ipv6 and set the family appropriately * Add test case for wsgi.Server and service.WSGIService Change-Id: I6edd467467fa3d623d62e146750b3a6c42d8833c
* | | | | | | | | | | | | | | Fix bash syntax error in run_tests.shAlessio Ababilov2013-01-101-1/+1
| |/ / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Spaces are not allowed in bash assignments. Change-Id: Ib80b10802259f5bff2ad0489f9efd763e5995056
* | | | | | | | | | | | | | Merge "Remove unused baremetal PXE options"Jenkins2013-01-101-11/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Remove unused baremetal PXE optionsMark McLoughlin2013-01-091-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These options were introduced by commit a2f260f and appear to never to have been used. Change-Id: Idf1387fd453b7b1c392db764934c148dcbfc5d66
* | | | | | | | | | | | | | | Merge "Remove unused opt import in scheduler.driver"Jenkins2013-01-101-1/+0
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Remove unused opt import in scheduler.driverMark McLoughlin2013-01-091-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reported by Joe Gordon. Change-Id: I8d48cad8f7f1548d20f0c3c0d8a4b11dcc351cf4
* | | | | | | | | | | | | | | Merge "Move global service networking opts to new module"Jenkins2013-01-1043-87/+106
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Move global service networking opts to new moduleMark McLoughlin2013-01-0943-87/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The my_ip, host and use_ipv6 options are used all over the codebase and they're pretty well related to each other. Create a new netconf module for them to live in. There are now no options registered globally in nova.config! blueprint: scope-config-opts Change-Id: Ifde37839ae6f38e6bf99dff1e80b8e25fd68ed25
* | | | | | | | | | | | | | | Merge "Move memcached_servers opt into common.memorycache"Jenkins2013-01-105-25/+26
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Move memcached_servers opt into common.memorycacheMark McLoughlin2013-01-095-25/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a factory function to nova.common.memorycache which consolidates the code to choose between real or fake memcache. This then means that memcached_servers is used in just one place and we can move the config option there. blueprint: scope-config-opts Change-Id: I67c191e0db58364eda4162b9e881606063509b9d
* | | | | | | | | | | | | | | Merge "Move service_down_time to nova.service"Jenkins2013-01-106-6/+7
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Move service_down_time to nova.serviceMark McLoughlin2013-01-096-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint: scope-config-opts Change-Id: I5fddb3768348c43a38b72dbf738b0c7fa2967691
* | | | | | | | | | | | | | | Merge "Move vpn_key_suffix into pipelib"Jenkins2013-01-104-6/+4
|\| | | | | | | | | | | | | | | |_|_|_|_|/ / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Move vpn_key_suffix into pipelibMark McLoughlin2013-01-094-6/+4
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apart from one use in the EC2 API, vpn_key_suffix is only used in pipelib. blueprint: scope-config-opts Change-Id: If90b9e77905161ea8d9263e8bae2c47ac5f79e91
* | | | | | | | | | | | | Merge "fix N402 on tools/"Jenkins2013-01-103-3/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | fix N402 on tools/Sean Dague2013-01-093-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix N402 (single line docstrings should end in a period) Change-Id: I2086dfb33a6efeb6ef6edc6a326182b6deda2527
* | | | | | | | | | | | | | Merge "fix N402 for nova-manage"Jenkins2013-01-101-21/+21
|\| | | | | | | | | | | | |
| * | | | | | | | | | | | | fix N402 for nova-manageSean Dague2013-01-091-21/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix N402 (docstrings should end in a period) for nova-manage Change-Id: Ia07796117fb9fce6c098a58aa86a6b47c6f7eda0
* | | | | | | | | | | | | | Merge "fix N402 for rest of nova"Jenkins2013-01-1013-37/+37
|\| | | | | | | | | | | | |
| * | | | | | | | | | | | | fix N402 for rest of novaSean Dague2013-01-0913-37/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix N402 (single line docstrings should end in a period) for rest of nova files Change-Id: I57d0d9ab01345dd83e544e476d79d2c2ca68ee51
* | | | | | | | | | | | | | Merge "fix N402 for nova/c*"Jenkins2013-01-1011-41/+41
|\| | | | | | | | | | | | |
| * | | | | | | | | | | | | fix N402 for nova/c*Sean Dague2013-01-0911-41/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix N402 (single line docstring comments should end in a period) for nova/cells, nova/compute, and nova/conductor Change-Id: Ia2d4e64f4dde346a26aeba8cf10dd2000585caa0