Skip to content

Commit bab6a91

Browse files
techniciantedfacebook-github-bot
authored andcommitted
Include command information for process spawn errors
Summary: In order to facilitate debugging, include full command information in the error message of spawn. This at least will provide clear indication of which executable it attempted to run with all of its argments. Reviewed By: mariusae, ahmadsharif1, vidhyav Differential Revision: D76853418 fbshipit-source-id: 928c7e7d364dce36c979cbebbd1d9a94a2130937
1 parent c14038c commit bab6a91

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

hyperactor_mesh/src/alloc/process.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -321,7 +321,7 @@ impl ProcessAlloc {
321321
match cmd.spawn() {
322322
Err(err) => {
323323
// Likely retry won't help here so fail permanently.
324-
let message = format!("spawn {}: {}", index, err);
324+
let message = format!("spawn index: {}, command: {:?}: {}", index, cmd, err);
325325
tracing::error!(message);
326326
self.failed = true;
327327
Some(ProcState::Failed {

0 commit comments

Comments
 (0)