Skip to content

Commit e9c7bc7

Browse files
authored
[autoscaler] Set new_config logging to debug in autoscaler (#38729)
Currently, it's really hard to debug autoscaler failures using monitor.out as it logs redundant new_config continuously in recent change. Fix is to move from print to DEBUG ubuntu@ip-10-0-143-38:~$ ls -lah /tmp/ray/session_latest/logs/monitor* -rw-rw-r-- 1 ubuntu ubuntu 2.2K Aug 22 01:19 /tmp/ray/session_latest/logs/monitor.err -rw-rw-r-- 1 ubuntu ubuntu 7.8M Aug 22 16:23 /tmp/ray/session_latest/logs/monitor.log -rw-rw-r-- 1 ubuntu ubuntu 124M Aug 22 16:23 /tmp/ray/session_latest/logs/monitor.out Related issue number "Closes #38727" --------- Signed-off-by: maheedhar reddy chappidi <chappidm@amazon.com>
1 parent 8db2a47 commit e9c7bc7

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

python/ray/autoscaler/_private/autoscaler.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1036,7 +1036,10 @@ def reset(self, errors_fatal=False):
10361036
"available until you upgrade ray on your cluster.",
10371037
exc_info=e,
10381038
)
1039-
print("new config", new_config, type(new_config))
1039+
logger.debug(
1040+
f"New config after validation: {new_config},"
1041+
f" of type: {type(new_config)}"
1042+
)
10401043
(new_runtime_hash, new_file_mounts_contents_hash) = hash_runtime_conf(
10411044
new_config["file_mounts"],
10421045
new_config["cluster_synced_files"],

0 commit comments

Comments
 (0)