Skip to content

Commit 538343f

Browse files
authored
[CI] Reduce logging level when checking if docker image exists (#17221)
Prior to this commit, the `image_exists` utility in `determine_docker_images.py` logged the full response for success, and the full HTTP error if an exception is caught. However, this is the expected behavior when loading a docker image from `tlcpackstaging`, such as the current images tagged with `20240428-060115-0b09ed018`. Logging this fallback as an error makes it difficult to find the first actual error that occurred in CI. This commit updates these logging statments `logging.info` and `logging.exception` to instead use `logging.debug`.
1 parent 16f8822 commit 538343f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ci/scripts/jenkins/determine_docker_images.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,11 @@ def image_exists(spec: str) -> bool:
6262
name, tag = spec.split(":")
6363
try:
6464
r = docker_api(f"repositories/{name}/tags/{tag}")
65-
logging.info(f"Image exists, got response: {json.dumps(r, indent=2)}")
65+
logging.debug(f"Image exists, got response: {json.dumps(r, indent=2)}")
6666
return True
6767
except urllib.error.HTTPError as e:
6868
# Image was not found
69-
logging.exception(e)
69+
logging.debug(e)
7070
return False
7171

7272

0 commit comments

Comments
 (0)