Skip to content

Commit ad3f3f5

Browse files
authored
Final SLURM fixes (meta-pytorch#398)
1 parent 4b3b3c2 commit ad3f3f5

File tree

2 files changed

+5
-11
lines changed

2 files changed

+5
-11
lines changed

src/forge/controller/provisioner.py

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -289,16 +289,10 @@ def bootstrap(env: dict[str, str]):
289289
for env_var in all_env_vars():
290290
env_vars[env_var.name] = str(env_var.get_value())
291291

292-
if MONARCH_HOSTMESH_V1.get_value():
293-
procs = host_mesh.spawn_procs(
294-
per_host={"procs": num_procs},
295-
setup=functools.partial(bootstrap, env=env_vars),
296-
)
297-
else:
298-
procs = host_mesh.spawn_procs(
299-
per_host={"procs": num_procs},
300-
bootstrap=functools.partial(bootstrap, env=env_vars),
301-
)
292+
procs = host_mesh.spawn_procs(
293+
per_host={"procs": num_procs},
294+
bootstrap=functools.partial(bootstrap, env=env_vars),
295+
)
302296

303297
if is_remote:
304298
await self.launcher.remote_setup(procs)

src/forge/env.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ def get_value(self) -> Any:
100100
)
101101

102102
MONARCH_HOSTMESH_V1 = EnvVar(
103-
name="MONARCH_HOSTMESH_V1",
103+
name="MONARCH_HOST_MESH_V1_REMOVE_ME_BEFORE_RELEASE",
104104
default=False,
105105
description="Whether or not to use Monarch's experimental hostmesh v1 APIs",
106106
)

0 commit comments

Comments
 (0)