sascha/workload-automation
1
0
Fork 0
mirror of https://github.com/ARM-software/workload-automation.git synced 2025-04-20 09:40:50 +01:00
Code Issues Releases Wiki Activity

fw/parsers: Ensure plug-in names are converted to an identifier

Browse Source 
Ensure that a plug-ins config entry is converted to an identifier before being
stored in the PluginCache so that the relevant configuration can be
retrieved appropriately. For example this allows for both 'trace-cmd' and
'trace_cmd' to be used as config entries to provide configuration for the
'trace-cmd' plugin.
This commit is contained in:
Marc Bonnici 2018-07-19 15:21:36 +01:00 committed by setrofim
parent fbfd81caeb
commit b729f7c9e4
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
wa/framework/configuration/parsers.py
View File

@ -18,6 +18,8 @@ import os
import logging import logging
from functools import reduce # pylint: disable=redefined-builtin from functools import reduce # pylint: disable=redefined-builtin
from devlib.utils.types import identifier
from wa.framework.configuration.core import JobSpec from wa.framework.configuration.core import JobSpec
from wa.framework.exception import ConfigError from wa.framework.exception import ConfigError
from wa.utils import log from wa.utils import log
@ -72,8 +74,8 @@ class ConfigParser(object):
for name, values in raw.items(): for name, values in raw.items():
# Assume that all leftover config is for a plug-in or a global # Assume that all leftover config is for a plug-in or a global
# alias it is up to PluginCache to assert this assumption # alias it is up to PluginCache to assert this assumption
logger.debug('Caching "{}" with "{}"'.format(name, values)) logger.debug('Caching "{}" with "{}"'.format(identifier(name), values))
state.plugin_cache.add_configs(name, values, source) state.plugin_cache.add_configs(identifier(name), values, source)
except ConfigError as e: except ConfigError as e:
if wrap_exceptions: if wrap_exceptions: