Branch: refs/heads/master
  Home:   https://github.com/uzh/gc3pie
  Commit: 5d6b1f0517129623694ee7f84d309876f74d347e
      
https://github.com/uzh/gc3pie/commit/5d6b1f0517129623694ee7f84d309876f74d347e
  Author: Sergio Maffioletti <sergio.maffiole...@gmail.com>
  Date:   2018-03-05 (Mon, 05 Mar 2018)

  Changed paths:
    M gc3libs/backends/shellcmd.py
    M gc3libs/backends/tests/test_shellcmd.py

  Log Message:
  -----------
  Make `ShellcmdLrms` handle `app.requested_memory` as total app memory.

`Application.requested_memory` is the *total* amount of memory requested by the 
application independently of the number of cores, as documented [in the 
`Application` docstring][1].

Now, different backends handle `Application.requested_memory` differently; see 
for example the behaviour of `Application.sbatch()` where 
`Application..requested_memory` is handled as a per-node requirement (see the 
documentation for the '--mem' option [in the SLURM man pages][2])

So the way to solve this problem is to make sure that 
`Application.requested_memory` is handled consistently for all backends, hence, 
change `ShellcmdLrms` accordingly.

[1]: https://github.com/uzh/gc3pie/blob/master/gc3libs/__init__.py#L765
[2]: https://slurm.schedmd.com/sbatch.html


-- 
You received this message because you are subscribed to the Google Groups 
"gc3pie-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to gc3pie-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/gc3pie-dev/5a9d21b65d6af_5a332b10c039bc0414654%40hookshot-fe-cace476.cp1-iad.github.net.mail.
For more options, visit https://groups.google.com/d/optout.

Reply via email to