openstack/nova
Code Issues Proposed changes

Fix missing versioned notification examples

Browse Source 
Python 3 is used in executing 'tox -e docs' by default currently.
When Python 3 is used, there are some missing notification examples.

In python 3, map function returns an iterator instead of a list,
and importlib.import_module is not executed in the document generation.
So it causes missing versioned notification examples in the nova docs.

This patch fixes it.

Change-Id: Ie4f3f9be0ca7f94ce00a14f3d825a067a807eb12
Closes-Bug: #1779606
This commit is contained in:
Takashi NATSUME 2018-07-02 13:59:19 +09:00
parent 4ea64cad3f
commit 54d3e7096c
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
doc/ext/versioned_notifications.py
View File

@ -56,9 +56,9 @@ jQuery(document).ready(function(){
return self._build_markup(notifications)
def _import_all_notification_packages(self):
map(lambda module: importlib.import_module(module),
('nova.notifications.objects.' + name for _, name, _ in
pkgutil.iter_modules(nova.notifications.objects.__path__)))
list(map(lambda module: importlib.import_module(module),
('nova.notifications.objects.' + name for _, name, _ in
pkgutil.iter_modules(nova.notifications.objects.__path__))))
def _collect_notifications(self):
self._import_all_notification_packages()