Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

Commit

Permalink
fixed a few places where repr isn't called for some reason
Browse files Browse the repository at this point in the history
  • Loading branch information
izapolsk committed Nov 26, 2019
1 parent 1771026 commit 381d0d0
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cfme/fixtures/parallelizer/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ def _slave_audit(self):
if returncode:
slave.process = None
if returncode == -9:
msg = f'{slave.id.decode("utf-8")} killed due to error, respawning'
msg = f'{slave.id} killed due to error, respawning'
else:
msg = f'{slave.id} terminated unexpectedly with status {returncode}, respawning'
if slave.tests:
Expand Down Expand Up @@ -291,6 +291,7 @@ def monitor_shutdown(self, slave):

def _monitor_shutdown_t(self, slaveid, process):
# a KeyError here means self.slaves got mangled, indicating a problem elsewhere
slaveid = slaveid.decode('utf-8') if isinstance(slaveid, bytes) else slaveid
if process is None:
self.log.warning('Slave was missing when trying to monitor shutdown')

Expand Down Expand Up @@ -361,7 +362,7 @@ def send_tests(self, slave):
test_perc = self.sent_tests * 100 / collect_len
self.print_message(
f'sent {tests_len} tests '
f'to {slave.id} \n'
f'to {slave.id.decode("utf-8") if isinstance(slave.id, bytes) else slave.id} \n'
f'Total sent: {self.sent_tests} of {collect_len}, ({test_perc:.1f}%)'
)
return tests
Expand Down

0 comments on commit 381d0d0

Please sign in to comment.