]> xenbits.xensource.com Git - people/andrewcoop/xen-test-framework.git/commitdiff
xtf-runner: python3 fix
authorAnthony PERARD <anthony.perard@citrix.com>
Thu, 17 Aug 2023 10:51:11 +0000 (11:51 +0100)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Fri, 18 Aug 2023 19:50:00 +0000 (20:50 +0100)
issue:
  File "/home/xtf/xtf-runner", line 410, in interpret_selection
    if not line.startswith("xen_caps"):
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: startswith first arg must be bytes or a tuple of bytes, not str

Adding `universal_newlines` open stdout as text file, so line should
be a `str`. `universal_newlines` is available on python 2.7. A new
alias `text` is only available in python 3.7.

Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
xtf-runner

index 6352a5b3e1268baf1f9a5986ff7ace4a5c397898..94ed1764d5414dbdc8ab5ecec298f6634aa17117 100755 (executable)
@@ -403,7 +403,7 @@ def interpret_selection(opts):
 
         host_envs = []
 
-        cmd = Popen(['xl', 'info'], stdout = PIPE)
+        cmd = Popen(['xl', 'info'], stdout = PIPE, universal_newlines = True)
         stdout, _ = cmd.communicate()
 
         for line in stdout.splitlines():