This repository has been archived on 2025-03-20. You can view files and clone it, but cannot push or open issues or pull requests.
sonic-buildimage/src/system-health
Joe LeVeque 2d77a36658
[system-health] Make run_command() Python 3-compliant (#6371)
Pass universal_newlines=True parameter to subprocess.Popen(); no longer use .encode('utf-8') on resulting stdout.
This was missed in #5886

Note: I would prefer to use text=True instead of universal_newlines=True, as the former is an alias only available in Python 3 and is more understandable than the latter. However, Even though the setup.py file for this package only specifies Python 3, the LGTM tool finds other Python 2 code in the repo and validates the code as Python 2 code and alerts that text=True is an invalid parameter. Will stick with universal_newlines=True for now. Once all Python code in the repo has been converted to Python 3, I will change all universal_newlines=True to text=True.
2021-01-07 05:48:13 -08:00
..
health_checker [system-health] Make run_command() Python 3-compliant (#6371) 2021-01-07 05:48:13 -08:00
scripts [system-health] Convert to Python 3 (#5886) 2020-12-29 14:04:09 -08:00
tests [system-health] Convert to Python 3 (#5886) 2020-12-29 14:04:09 -08:00
.gitignore [system-health] Update .gitignore file (#5688) 2020-10-22 11:58:27 -07:00
pytest.ini [system-health] Add support for monitoring system health (#4835) 2020-10-12 11:12:49 +03:00
setup.cfg [system-health] Add support for monitoring system health (#4835) 2020-10-12 11:12:49 +03:00
setup.py [system-health] Convert to Python 3 (#5886) 2020-12-29 14:04:09 -08:00