Skip to content

Commit d94f34d

Browse files
committed
[monarch] hyperactor: make sure that mock MAST jobs are unique
Without this, we cannot guarantee that the various world/proc names are unique. This was not a problem before D79478197 because each mesh was private -- the underlying agents were not wired together. However, since the mock MAST allocator uses the LocalAllocator to allocate host-local procs, they are now all wired together, and messages, and subsequently, we have overlapping routing entries. Differential Revision: [D79596215](https://our.internmc.facebook.com/intern/diff/D79596215/) ghstack-source-id: 300660703 Pull Request resolved: #756
1 parent cc8c88d commit d94f34d

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

hyperactor_mesh/src/proc_mesh/mesh_agent.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -210,6 +210,7 @@ impl MeshAgentMessageHandler for MeshAgent {
210210
let client = MailboxClient::new(channel::dial(forwarder)?);
211211
let default = super::global_router().fallback(client.into_boxed());
212212
let router = DialMailboxRouter::new_with_default(default.into_boxed());
213+
// let router = DialMailboxRouter::new_with_default(client.into_boxed());
213214
for (proc_id, addr) in address_book {
214215
router.bind(proc_id.into(), addr);
215216
}

0 commit comments

Comments
 (0)