diff options
author | Gwenael Lambrouin <gwenael.lambrouin@orange.com> | 2021-08-24 11:42:26 +0200 |
---|---|---|
committer | Gwenael Lambrouin <gwenael.lambrouin@orange.com> | 2021-08-24 14:41:58 +0200 |
commit | 58a44a8ea1e48ba8936485086794cb59795a2d03 (patch) | |
tree | d91beeac3df5c0ecef37eb093a5eff52c6317e37 /nfvbench/traffic_gen/trex_gen.py | |
parent | 1574b1655e47e4c37dfaf8de8cb0a8bfcb999894 (diff) |
Be explicit about text file encoding
Python PEP 597 (https://www.python.org/dev/peps/pep-0597) recommends to use an
explicit encoding for text files instead of the default locale encoding.
Pylint 2.10 adds a new checker named unspecified-encoding for that.
The present patch adds explicit utf-8 encoding to open() calls in nfvbench and
fixes pylint unspecified-encoding warnings.
Remark: this patch does not change nfvbench behaviour on systems where utf-8 is
the locale encoding, which is generally the case on Linux systems.
Change-Id: Ic4dfb37e1ea958452a0173f7630a68f0d95071ae
Signed-off-by: Gwenael Lambrouin <gwenael.lambrouin@orange.com>
Diffstat (limited to 'nfvbench/traffic_gen/trex_gen.py')
-rw-r--r-- | nfvbench/traffic_gen/trex_gen.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/nfvbench/traffic_gen/trex_gen.py b/nfvbench/traffic_gen/trex_gen.py index 41768b1..dff72ac 100644 --- a/nfvbench/traffic_gen/trex_gen.py +++ b/nfvbench/traffic_gen/trex_gen.py @@ -761,7 +761,7 @@ class TRex(AbstractTrafficGenerator): after = None last = None try: - with open('/tmp/trex.log', 'r') as trex_log: + with open('/tmp/trex.log', 'r', encoding="utf-8") as trex_log: for _line in trex_log: line = _line.strip() if line.startswith('Usage:'): @@ -912,7 +912,7 @@ class TRex(AbstractTrafficGenerator): break last_size = size time.sleep(1) - with open(logpath, 'r') as f: + with open(logpath, 'r', encoding="utf-8") as f: message = f.read() else: message = e.message |