Skip to content

Commit

Permalink
Pylint: enable missing-timeout
Browse files Browse the repository at this point in the history
  • Loading branch information
avylove committed Feb 21, 2023
1 parent 4bd803f commit 439b981
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
6 changes: 4 additions & 2 deletions lisa/sut_orchestrator/azure/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -1134,7 +1134,9 @@ def save_console_log(
)
if saved_path:
screenshot_raw_name = saved_path / f"{screenshot_file_name}.bmp"
screenshot_response = requests.get(diagnostic_data.console_screenshot_blob_uri)
screenshot_response = requests.get(
diagnostic_data.console_screenshot_blob_uri, timeout=60
)
screenshot_raw_name.write_bytes(screenshot_response.content)
try:
with Image.open(screenshot_raw_name) as image:
Expand All @@ -1148,7 +1150,7 @@ def save_console_log(
)
screenshot_raw_name.unlink()

log_response = requests.get(diagnostic_data.serial_console_log_blob_uri)
log_response = requests.get(diagnostic_data.serial_console_log_blob_uri, timeout=60)
if log_response.status_code == 404:
log.debug(
"The serial console is not generated. "
Expand Down
1 change: 0 additions & 1 deletion pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ disable=
inconsistent-return-statements,
invalid-overridden-method,
keyword-arg-before-vararg,
missing-timeout,
modified-iterating-list,
no-member,
redefined-argument-from-local,
Expand Down

0 comments on commit 439b981

Please sign in to comment.