openstack
/
openstack-doc-tools
Code Issues Proposed changes

Merge "Adds multiple group support for config options in autohelp.py"

This commit is contained in:
Jenkins 2013-12-12 07:56:11 +00:00 committed by Gerrit Code Review
parent 53f6a4270c ccde3f5a3f
commit a5cb900fc8
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
autogenerate-config-docs/README.md
View File

@ -54,14 +54,14 @@ eg
2) Grouping of flags
This is currently done manually, by using the flag name file and placing
a category after a space.
one or more categories after a space.
eg
$ head flagmappings/glance.flagmappings
admin\_password registry
admin\_password registry api
admin\_role api
admin\_tenant\_name registry
admin\_tenant\_name registry api
admin\_user registry
...

8
autogenerate-config-docs/common.py
View File

@ -37,18 +37,20 @@ def populate_groups(filepath):
"""
Takes a file formatted with lines of config option and group
separated by a space and constructs a dictionary indexed by
group, which is returned..
group, which is returned.
"""
groups = defaultdict(list)
groups_file = open(os.path.expanduser(filepath), 'r')
for line in groups_file:
try:
option, group = line.split(None, 1)
option, group_list = line.split(None, 1)
group_list = group_list.split(None)
except ValueError:
print "Couldn't read groups file line:%s" % line
print "Check for formatting errors - did you add the group?"
sys.exit(1)
groups[group.strip()].append(option)
for group in group_list:
groups[group.strip()].append(option)
return groups