python: fix flake8 warnings and format with black

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I90fe7f4eccb9014245225b6bc61183d7455b0fae

regulators: Add device_id to validation tool

Enhance the configuration file validation tool to verify that any
"device_id" properties refer to a valid device ID. This property exists
in the new pha

regulators: Add device_id to validation tool

Enhance the configuration file validation tool to verify that any
"device_id" properties refer to a valid device ID. This property exists
in the new phase_fault_detection object.

Add the following new automated tests
* Test where phase_fault_detection object specifies an invalid rule_id
* Test where phase_fault_detection object specifies an invalid device_id

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: I242ced0a2d0c4483d0dd7c4d36781a9f588a7e4f

show more ...

regulators: Improve config file validator

Improve error checking in config file validator.

Add checking if config file does not exist, config file is not readable,
and config file is not a JSON fil

regulators: Improve config file validator

Improve error checking in config file validator.

Add checking if config file does not exist, config file is not readable,
and config file is not a JSON file.
Add checking if schema file does not exist, schema file is not readable,
and schema file is not a JSON file.

Tested:
Run local CI with -Dlong-tests=enabled to enable tests for
validate-regulators-config.py.

1/7 phosphor-regulators-tests OK 32.49s

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I5a84949e89be004a20a04c00d16b49286cb42f37

show more ...

Use python3

python2 is EOL, specify the python scripts to use python3.

Tested: Verify it builds fine in local CI. With bitbake recipe changes,
verify it builds OK with python3native.

Signe

Use python3

python2 is EOL, specify the python scripts to use python3.

Tested: Verify it builds fine in local CI. With bitbake recipe changes,
verify it builds OK with python3native.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: Icd5d10193e9323bbdbcd854e9027e014e01a1829

show more ...

regulators: Add check_number_of_elements_in_masks function.

Add check_number_of_elements_in_masks function to the config file
validation tool.
It checks if the number of bit masks in the 'masks' pro

regulators: Add check_number_of_elements_in_masks function.

Add check_number_of_elements_in_masks function to the config file
validation tool.
It checks if the number of bit masks in the 'masks' property matches
the number of byte values in the 'values' property.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Ie33d78867b0c3ba508ff3693fe5969596d70e95a

show more ...

regulators: Add check_rule_id_exists function.

Add check_rule_id_exists function to the config file validation tool.
It checks if a rule_id property specifies a rule ID that does not
exist.

Signed-

regulators: Add check_rule_id_exists function.

Add check_rule_id_exists function to the config file validation tool.
It checks if a rule_id property specifies a rule ID that does not
exist.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I960a1b27a74b89b3bfb89ec18ad54dbd25c9388e

show more ...

regulators: Add check_set_device_value_exists function.

Add check_set_device_value_exists function to the config file validation
tool.
It checks if a set_device action specifies a device ID that doe

regulators: Add check_set_device_value_exists function.

Add check_set_device_value_exists function to the config file validation
tool.
It checks if a set_device action specifies a device ID that does not
exist.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I95e6f9b51cb2c75c3d95b3ca1c4377f4b6e0171c

show more ...

regulators: Add check_run_rule_value_exists function.

Add check_run_rule_value_exists function to the config file validation tool.
It checks if any run_rule actions specify a rule ID that does not e

regulators: Add check_run_rule_value_exists function.

Add check_run_rule_value_exists function to the config file validation tool.
It checks if any run_rule actions specify a rule ID that does not exist.
Add get_values function.
It finds all occurrences of a key within the config file and returns the
associated values.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I165426f10fcda052340a7a979228365bb7f1602d

show more ...

regulators: Add check infinite loops function.

Add check_infinite_loops function.
It checks whether infinite loop happened in the config file.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-

regulators: Add check infinite loops function.

Add check_infinite_loops function.
It checks whether infinite loop happened in the config file.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I34e7feb6e6200687538c1a68ba7e3cb785055d6e

show more ...

regulators: Add check_duplicate_object_id function.

Add the check_duplicate_object_id function to the config file validation tool.
It checks that there aren't any JSON objects with the same 'id' pro

regulators: Add check_duplicate_object_id function.

Add the check_duplicate_object_id function to the config file validation tool.
It checks that there aren't any JSON objects with the same 'id' property value.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I1e0d8de50f3687f88509476a5cc81e5577d4c84c

show more ...

regulators: Add check duplicate ID function.

Add check duplicate id and number functions.
check_duplicate_rule_id
check_duplicate_chassis_number
check_duplicate_device_id
check_duplicate_rail_id

Si

regulators: Add check duplicate ID function.

Add check duplicate id and number functions.
check_duplicate_rule_id
check_duplicate_chassis_number
check_duplicate_device_id
check_duplicate_rail_id

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Ic7874037ee46614ca34f88113adc4d246608b718

show more ...

Add validate-regulators-config.py validation tool.

validate-regulators-config.py is a tool that validates configuration
files for the phosphor-regulators application. See
phosphor-regulators/docs/co

Add validate-regulators-config.py validation tool.

validate-regulators-config.py is a tool that validates configuration
files for the phosphor-regulators application. See
phosphor-regulators/docs/config_file/README.md for the
command to use the tool.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I41cd4070dbb6ec27e834308a9eb0a819feb77319

show more ...