Merge pull request #7609 from hppritcha/topic/fix_for_issue5115

debuggers: don't remove session dirctory

orte/mca/state/base/state_base_fns.c

@@ -3,6 +3,7 @@
  * Copyright (c) 2014-2018 Intel, Inc. All rights reserved.
  * Copyright (c) 2018      Research Organization for Information Science
  *                         and Technology (RIST).  All rights reserved.
+ * Copyrigth (c) 2020      Triad National Security, LLC. All rights reserved.
  * $COPYRIGHT$
  *
  * Additional copyrights may follow
@@ -734,9 +735,13 @@ void orte_state_base_track_procs(int fd, short argc, void *cbdata)
             /* Clean up the session directory as if we were the process
              * itself.  This covers the case where the process died abnormally
              * and didn't cleanup its own session directory.
+             * Don't do this for debugger daemons otherwise reattach is
+             * broken.  See https://github.com/open-mpi/ompi/issues/5115.
              */
+            if (!ORTE_FLAG_TEST(jdata, ORTE_JOB_FLAG_DEBUGGER_DAEMON)) {
                 orte_session_dir_finalize(proc);
             }
+        }
         /* if we are trying to terminate and our routes are
          * gone, then terminate ourselves IF no local procs
          * remain (might be some from another job)