Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion checkbox-ng/checkbox_ng/launcher/agent.py
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,8 @@ def invoked(self, ctx):
protocol_config={
"allow_all_attrs": True,
"allow_setattr": True,
"sync_request_timeout": 1,
# this is the max server to client attr accessing speed,
"sync_request_timeout": 30,
"propagate_SystemExit_locally": True,
},
)
Expand Down
6 changes: 5 additions & 1 deletion checkbox-ng/checkbox_ng/launcher/controller.py
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,11 @@
def connect_and_run(self, host, port=18871):
config = rpyc.core.protocol.DEFAULT_CONFIG.copy()
config["allow_all_attrs"] = True
config["sync_request_timeout"] = 120
# this is the max client to server attr accessing speed, unbounded (-1)
# to avoid timing out on any attr request on an enstablished call. This
# way even particularly slow @properties won't fail
config["sync_request_timeout"] = -1

Check warning on line 233 in checkbox-ng/checkbox_ng/launcher/controller.py

View check run for this annotation

Codecov / codecov/patch

checkbox-ng/checkbox_ng/launcher/controller.py#L233

Added line #L233 was not covered by tests

keep_running = False
server_msg = None
self._prepare_transports()
Expand Down
6 changes: 3 additions & 3 deletions checkbox-ng/checkbox_ng/launcher/stages.py
Original file line number Diff line number Diff line change
Expand Up @@ -762,16 +762,16 @@
_logger.error(
_(
"Problem with a '%s' report using '%s' exporter "
"sent to '%s' transport. Reason %s"
"sent to '%s' transport. Reason: %s"
),
name,
exporter_id,
transport.url,
exc,
repr(exc),
)
import traceback

traceback.print_tb(exc)
traceback.print_exception(exc)

Check warning on line 774 in checkbox-ng/checkbox_ng/launcher/stages.py

View check run for this annotation

Codecov / codecov/patch

checkbox-ng/checkbox_ng/launcher/stages.py#L774

Added line #L774 was not covered by tests

self._reset_auto_submission_retries()
done_sending = True
Expand Down
Loading