Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LLNL STAT tool not working with v2.0.2 #2411

Closed
jsquyres opened this issue Nov 14, 2016 · 3 comments
Closed

LLNL STAT tool not working with v2.0.2 #2411

jsquyres opened this issue Nov 14, 2016 · 3 comments
Assignees
@rhc54
Labels
bug
Milestone
v2.0.2

Comments

@jsquyres
Copy link
Member

Per verbal report from @rhc54 on the show floor here at SC, apparently STAT is not working properly with v2.0.2. Need to get this fixed.
@jsquyres jsquyres added this to the v2.0.2 milestone Nov 14, 2016
@rhc54 rhc54 mentioned this issue Nov 16, 2016
Merged
@hppritcha
Copy link
Member

Per discussion at the devel core meeting on 11/22 remove blocker label. Can be fixed in a 2.0.3 release.

@hppritcha
Copy link
Member

removing blocker label per devel core discussion 11/22.

@jsquyres
Copy link
Member Author

Looks like this was fixed in #2425.

markalle pushed a commit to markalle/ompi that referenced this issue Sep 12, 2020
@gpaulsen
Cherry-pick from:
7e180d2 
--------------------------
|From 2977194 Mon Sep 17 00:00:00 2001
|From: Ralph Castain <rhc@open-mpi.org>
|Date: Tue, 15 Nov 2016 19:27:26 -0700
|Subject: [PATCH] Fix debugger attach and cospawn of debugger daemons for the
| STAT debugger. Add ability to test the support minus the actual debugger.
|
|Fixes open-mpi#2411
|
|Signed-off-by: Ralph Castain <rhc@open-mpi.org>
|---

(cherry picked from commit 5cb312a425d55bf2e1e343d9d9c3c35fa428128a)
markalle pushed a commit to markalle/ompi that referenced this issue Sep 12, 2020
@gpaulsen
Fix debugger attach and cospawn of debugger daemons for the STAT debu…
3d4e193 
…gger. Add ability to test the support minus the actual debugger.

Fixes open-mpi#2411

Continue cleanup of STAT debugger attach:

* Limit the number of times we retry sending of a message to avoid an infinite loop

* Don't execute the "init_debugger_after_spawn" state for debugger jobs

* Add a new test program "attach" that takes the debugger attach fifo as its argument, and then simulates attach by writing a byte down the fifo

Output the attach fifo info if we are testing attach so we know where to attach to - otherwise, use the output_verbose

Always send "debugger release" to the job actually being debugged, not the debugger itself

Signed-off-by: Ralph Castain <rhc@open-mpi.org>

Remove debug

Signed-off-by: Ralph Castain <rhc@open-mpi.org>

Conflicts:
	orte/tools/orterun/orterun.c

(cherry picked from commit 8fa33bfdb3f6ca1667c558f06c3c8f0fb5482a8d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rhc54 rhc54

Labels
bug
Projects
None yet
Milestone
v2.0.2
Development

No branches or pull requests
3 participants
@jsquyres @rhc54 @hppritcha