[Chore][Autoscaler] Clarify disable_launch_config_check comment in StandardAutoscaler (#51751)

nadongjun · web-flow · commit 84b12932e529 · 2025-03-27T13:21:13.000-07:00
Signed-off-by: Dongjun Na &lt;kmu5544616@gmail.com&gt;
diff --git a/python/ray/autoscaler/_private/autoscaler.py b/python/ray/autoscaler/_private/autoscaler.py
@@ -279,9 +279,10 @@ def read_fn():
         )
         logger.info(f"{DISABLE_NODE_UPDATERS_KEY}:{self.disable_node_updaters}")
 
-        # Disable launch config checking if true.
-        # This is set in the fake_multinode situations where there isn't any
-        # meaningful node "type" to enforce.
+        # Disable launch configuration checking if set to true.
+        # This setting is used in scenarios where there is no meaningful node type
+        # to enforce, such as in fake multinode situations. When this option is enabled,
+        # outdated nodes will not be terminated.
         self.disable_launch_config_check = self.config["provider"].get(
             DISABLE_LAUNCH_CONFIG_CHECK_KEY, False
         )

Original file line number	Diff line number	Diff line change
`@@ -279,9 +279,10 @@ def read_fn():`
`279`	`279`	`)`
`280`	`280`	`logger.info(f"{DISABLE_NODE_UPDATERS_KEY}:{self.disable_node_updaters}")`
`281`	`281`
`282`		`- # Disable launch config checking if true.`
`283`		`- # This is set in the fake_multinode situations where there isn't any`
`284`		`- # meaningful node "type" to enforce.`
	`282`	`+ # Disable launch configuration checking if set to true.`
	`283`	`+ # This setting is used in scenarios where there is no meaningful node type`
	`284`	`+ # to enforce, such as in fake multinode situations. When this option is enabled,`
	`285`	`+ # outdated nodes will not be terminated.`
`285`	`286`	`self.disable_launch_config_check = self.config["provider"].get(`
`286`	`287`	`DISABLE_LAUNCH_CONFIG_CHECK_KEY, False`
`287`	`288`	`)`