> -----Original Message-----
> From: Juraj Linkeš <[email protected]>
> Sent: Friday, July 29, 2022 6:56 PM
> To: [email protected]; [email protected]; Randles, Ronan
> <[email protected]>; [email protected];
> [email protected]; Tu, Lijuan <[email protected]>
> Cc: [email protected]; Juraj Linkeš <[email protected]>
> Subject: [PATCH v4 6/9] dts: add config parser module
> 
> From: Owen Hilyard <[email protected]>
> 
> The configuration is split into two parts, one defining the parameters of the 
> test
> run and the other defining the topology to be used.
> 
> The format of the configuration is YAML. It is validated according to a json
> schema which also servers as detailed documentation of the various
> configuration fields. This means that the complete set of allowed values are 
> tied
> to the schema as a source of truth. This enables making changes to parts of 
> DTS
> that interface with config files without a high risk of breaking someone's
> configuration.
> 
> This configuration system uses immutable objects to represent the 
> configuration,
> making IDE/LSP autocomplete work properly.
> 
> There are two ways to specify the configuration file path, an environment
> variable or a command line argument, applied in that order.
> 
> Signed-off-by: Owen Hilyard <[email protected]>
> Signed-off-by: Juraj Linkeš <[email protected]>
> ---
>  dts/conf.yaml                              |  7 ++
>  dts/framework/config/__init__.py           | 99 ++++++++++++++++++++++
>  dts/framework/config/conf_yaml_schema.json | 73 ++++++++++++++++
>  dts/framework/exception.py                 | 14 +++
>  dts/framework/settings.py                  | 65 ++++++++++++++
>  5 files changed, 258 insertions(+)
>  create mode 100644 dts/conf.yaml
>  create mode 100644 dts/framework/config/__init__.py  create mode 100644
> dts/framework/config/conf_yaml_schema.json
>  create mode 100644 dts/framework/settings.py
> 

Reviewed-by: Lijuan Tu <[email protected]>

Thanks,
Lijuan

Reply via email to