1. 29 Nov, 2012 1 commit
  2. 28 Nov, 2012 2 commits
  3. 27 Nov, 2012 5 commits
  4. 26 Nov, 2012 2 commits
  5. 22 Nov, 2012 1 commit
  6. 21 Nov, 2012 3 commits
  7. 20 Nov, 2012 3 commits
  8. 19 Nov, 2012 3 commits
  9. 09 Nov, 2012 2 commits
  10. 08 Nov, 2012 2 commits
  11. 07 Nov, 2012 5 commits
  12. 05 Nov, 2012 2 commits
  13. 02 Nov, 2012 3 commits
  14. 31 Oct, 2012 1 commit
  15. 29 Oct, 2012 2 commits
    • Morris Jette's avatar
      Fix bug with topology/tree and job with min-max node count. · e15cab3f
      Morris Jette authored
      Now try to get max node count rather than minimizing leaf switches used.
      For example, if each leaf switch has 8 nodes then a request for -N4-16
      would allocate 8 nodes (one leaf switch) rather than 16 nodes over two
      leaf switches.
      e15cab3f
    • Morris Jette's avatar
      Cray - Prevent calling basil_confirm more than once per job using a flag. · faa96d55
      Morris Jette authored
          Anyhow, after applying the patch, I was still running into the same difficulty.  Upon a closer look, I saw that I was still receiving the ALPS backend error in the slurmctld.log file.  When I examined the code pertaining this and ran some SLURM-independent tests, I found that we were executing the do_basil_confirm function multiple times in the cases where it would fail.  My independent tests show precisely the same behaviour; that is, if you make a reservation request, then successfully confirm it and then attempt to confirm it again, you receive this error message.  However, the "apstat -rvv" command shows that the ALPS reservation is fine and therefore I concluded that this particular ALPS/BASIL message is more of an informational one and not a "show-stopper."  In other words, I can consider the node ready at this point.
          As a simple work around, I currently just inserted an if-block immediately after the call to "basil_confirm" in function "do_basil_confirm" in ".../src/plugins/select/cray/basil_interface.c."  The if-statment checks for "BE_BACKEND" and if this is the result then it prints an informational message to slurmctld.log and sets the variable rc=0 so that we can consider the node ready.  This, now allows my prolog scripts to run and I can clearly see the SLURM message that I had placed in that if-block.
           However, I am not certain if we really should just allow this error code to pass through as it seems like it could be a fairly generic code and there could be various other causes of it where we would not wish to allow it to pass.  I really only want to limit the number of calls to basil_confirm to one.  Perhaps I could add a field to the job_record so that I can mark whether the ALPS reservation had been confirmed or not.
      faa96d55
  16. 26 Oct, 2012 2 commits
  17. 25 Oct, 2012 1 commit