openstack/deb-python-dcos
Code Issues Proposed changes

Fixed bug with listing installed subcommands.

Browse Source 
The command names of installed subcommands are now embedded under a
'command' key instead of living at the top level. This commit fixes the
'merge_installed()' function to index into the dictionary of subcommands
properly to extract the names now that this change has been introduced.
This commit is contained in:
Kevin Klues
2017-06-19 21:13:34 -07:00
parent 9be20d45f1
commit 719a0450a4
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
dcos/package.py
View File

@@ -156,7 +156,8 @@ def merge_installed(apps, subcommands, app_only, cli_only):
del pkg['appId']
indexed_apps[name] = pkg
indexed_subcommands = {subcmd['name']: subcmd for subcmd in subcommands}
indexed_subcommands = {subcmd['command']['name']: subcmd
for subcmd in subcommands}
merged = []
for name, app in indexed_apps.items():