unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#31080] [PATCH 0/2] Add launchmon and spindle
@ 2018-04-06 16:02 Rouby Pierre-Antoine
  2018-04-09  6:58 ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
                   ` (2 more replies)
  0 siblings, 3 replies; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-06 16:02 UTC (permalink / raw)
  To: 31080; +Cc: Rouby Pierre-Antoine

This patch add package for launchmon and spindle, they compile and install
without any error. But not sure they binary works nice. If someone familiar
with spindle and launchmon could have a look ?

(note: the headers specify the lgpl2.1 only)

Rouby Pierre-Antoine (2):
  gnu: Add launchmon.
  gnu: Add spindle.

 gnu/packages/admin.scm | 63 +++++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 62 insertions(+), 1 deletion(-)

-- 
2.16.1

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 1/2] gnu: Add launchmon.
  2018-04-06 16:02 [bug#31080] [PATCH 0/2] Add launchmon and spindle Rouby Pierre-Antoine
@ 2018-04-09  6:58 ` Rouby Pierre-Antoine
  2018-04-09  6:59 ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
  2018-04-09 13:50 ` [bug#31080] [PATCH 0/2] Add launchmon and spindle Ludovic Courtès
  2 siblings, 0 replies; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-09  6:58 UTC (permalink / raw)
  To: 31080; +Cc: Rouby Pierre-Antoine

* gnu/packages/admin.scm (launchmon): New variable.
---
 gnu/packages/admin.scm | 34 +++++++++++++++++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 6a7bed389..4403f2a3b 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -90,7 +90,10 @@
   #:use-module (gnu packages gnome)
   #:use-module (gnu packages kerberos)
   #:use-module (gnu packages gtk)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages boost)
+  #:use-module (gnu packages elf)
+  #:use-module (gnu packages mpi))
 
 (define-public aide
   (package
@@ -2505,3 +2508,32 @@ printed instead of after the entire file has been read, which is often too
 late.")
     (home-page "https://jwilk.net/software/hungrycat")
     (license license:expat)))
+
+(define-public launchmon
+  (package
+    (name "launchmon")
+    (version "1.0.2")
+    (source (origin
+              (file-name (git-file-name name version))
+              (method url-fetch)
+              (uri (string-append
+                    "https://github.com/LLNL/LaunchMON/releases/download/v"
+                    version "/" name "-v" version ".tar.gz"))
+              (sha256
+               (base32
+                "0fm3nd9mydm9v2bf7bh01dbgrfnpwkapxa3dsvy3x1z0rz61qc0x"))))
+    (build-system gnu-build-system)
+    (inputs `(("munge" ,munge)
+              ("boost" ,boost)
+              ("libelf" ,libelf)
+              ("openmpi" ,openmpi)
+              ("libgcrypt" ,libgcrypt)
+              ("libgpg-error" ,libgpg-error)))
+    (synopsis "Software infrastructure for HPC environments")
+    (description
+     "LaunchMON is a software infrastructure that enables HPC
+(High-Performance Computing) run-time tools to co-locate tool daemons with a
+parallel job.  Its API allows a tool to identify all the remote processes of a
+job and to scalably launch daemons into the relevant nodes.")
+    (home-page "https://github.com/LLNL/LaunchMON")
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-06 16:02 [bug#31080] [PATCH 0/2] Add launchmon and spindle Rouby Pierre-Antoine
  2018-04-09  6:58 ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
@ 2018-04-09  6:59 ` Rouby Pierre-Antoine
  2018-04-09 14:06   ` Jonathan Brielmaier
  2018-04-09 13:50 ` [bug#31080] [PATCH 0/2] Add launchmon and spindle Ludovic Courtès
  2 siblings, 1 reply; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-09  6:59 UTC (permalink / raw)
  To: 31080; +Cc: Rouby Pierre-Antoine

* gnu/packages/admin.scm (spindle): New variable.
---
 gnu/packages/admin.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 4403f2a3b..c89662fc9 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -2537,3 +2537,32 @@ parallel job.  Its API allows a tool to identify all the remote processes of a
 job and to scalably launch daemons into the relevant nodes.")
     (home-page "https://github.com/LLNL/LaunchMON")
     (license license:lgpl2.1)))
+
+(define-public spindle
+  (package
+    (name "spindle")
+    (version "0.10")
+    (source (origin
+              (file-name (git-file-name name version))
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/hpc/Spindle.git")
+                    (commit "ff922c1df167c28ab312d02217ed7be65ec0e831")))
+              (sha256
+               (base32
+                "15n3ay0qq81r5v7fif61q1vdjcq44pp2nynkh3fvbzc9fj3c39wd"))))
+    (build-system gnu-build-system)
+    (arguments '(#:configure-flags '("--enable-sec-launchmon"
+                                     "--enable-sec-munge"
+                                     "--enable-sec-none")))
+    (inputs `(("munge" ,munge)
+              ("openmpi" ,openmpi)
+              ("launchmon" ,launchmon)
+              ("libgcrypt" ,libgcrypt)))
+    (synopsis "Scalable dynamic library and Python loading in HPC
+environments")
+    (description
+     "Spindle is a tool for improving the performance of dynamic library and
+Python loading in HPC environments.")
+    (home-page "https://github.com/hpc/Spindle")
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 0/2] Add launchmon and spindle
  2018-04-06 16:02 [bug#31080] [PATCH 0/2] Add launchmon and spindle Rouby Pierre-Antoine
  2018-04-09  6:58 ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
  2018-04-09  6:59 ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
@ 2018-04-09 13:50 ` Ludovic Courtès
  2018-04-10 15:20   ` Eric Bavier
  2 siblings, 1 reply; 18+ messages in thread
From: Ludovic Courtès @ 2018-04-09 13:50 UTC (permalink / raw)
  To: Rouby Pierre-Antoine; +Cc: Eric Bavier, 31080

Hi there!

I should say that this is Pierre-Antoine’s first week as an intern here
at Inria to work on Guix for HPC.  So welcome, Pierre-Antoine!  :-)

The patches look good to me, but as Pierre-Antoine wrote, we’d like to
get feedback from someone more familiar with the tools.  Eric maybe?

Thanks,
Ludo’.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-09  6:59 ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
@ 2018-04-09 14:06   ` Jonathan Brielmaier
  2018-04-09 20:34     ` Ludovic Courtès
  2018-04-10 10:03     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
  0 siblings, 2 replies; 18+ messages in thread
From: Jonathan Brielmaier @ 2018-04-09 14:06 UTC (permalink / raw)
  To: 31080

On 09/04/18 08:59, Rouby Pierre-Antoine wrote:
> * gnu/packages/admin.scm (spindle): New variable.
> ---
>  gnu/packages/admin.scm | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
> 
> diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
> index 4403f2a3b..c89662fc9 100644
> --- a/gnu/packages/admin.scm
> +++ b/gnu/packages/admin.scm

Welcome Pierre-Antoine! Please also include your name in the license
section at the top of "admin.scm".

~Jonathan

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-09 14:06   ` Jonathan Brielmaier
@ 2018-04-09 20:34     ` Ludovic Courtès
  2018-04-10 10:03     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
  1 sibling, 0 replies; 18+ messages in thread
From: Ludovic Courtès @ 2018-04-09 20:34 UTC (permalink / raw)
  To: Jonathan Brielmaier; +Cc: 31080

Jonathan Brielmaier <jonathan.brielmaier@web.de> skribis:

> On 09/04/18 08:59, Rouby Pierre-Antoine wrote:
>> * gnu/packages/admin.scm (spindle): New variable.
>> ---
>>  gnu/packages/admin.scm | 29 +++++++++++++++++++++++++++++
>>  1 file changed, 29 insertions(+)
>> 
>> diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
>> index 4403f2a3b..c89662fc9 100644
>> --- a/gnu/packages/admin.scm
>> +++ b/gnu/packages/admin.scm
>
> Welcome Pierre-Antoine! Please also include your name in the license
> section at the top of "admin.scm".

Oops, indeed.

Ludo’.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 1/2] gnu: Add launchmon.
  2018-04-09 14:06   ` Jonathan Brielmaier
  2018-04-09 20:34     ` Ludovic Courtès
@ 2018-04-10 10:03     ` Rouby Pierre-Antoine
  2018-04-10 10:03       ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
  1 sibling, 1 reply; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-10 10:03 UTC (permalink / raw)
  To: 31080; +Cc: Rouby Pierre-Antoine

* gnu/packages/admin.scm (launchmon): New variable.
---
 gnu/packages/admin.scm | 35 ++++++++++++++++++++++++++++++++++-
 1 file changed, 34 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 6a7bed389..bf50a51f4 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -20,6 +20,7 @@
 ;;; Copyright © 2017 Christopher Allan Webber <cwebber@dustycloud.org>
 ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
 ;;; Copyright © 2018 Arun Isaac <arunisaac@systemreboot.net>
+;;; Copyright © 2018 Pierre-Antoine Rouby <pierre-antoine.rouby@inria.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -90,7 +91,10 @@
   #:use-module (gnu packages gnome)
   #:use-module (gnu packages kerberos)
   #:use-module (gnu packages gtk)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages boost)
+  #:use-module (gnu packages elf)
+  #:use-module (gnu packages mpi))
 
 (define-public aide
   (package
@@ -2505,3 +2509,32 @@ printed instead of after the entire file has been read, which is often too
 late.")
     (home-page "https://jwilk.net/software/hungrycat")
     (license license:expat)))
+
+(define-public launchmon
+  (package
+    (name "launchmon")
+    (version "1.0.2")
+    (source (origin
+              (file-name (git-file-name name version))
+              (method url-fetch)
+              (uri (string-append
+                    "https://github.com/LLNL/LaunchMON/releases/download/v"
+                    version "/" name "-v" version ".tar.gz"))
+              (sha256
+               (base32
+                "0fm3nd9mydm9v2bf7bh01dbgrfnpwkapxa3dsvy3x1z0rz61qc0x"))))
+    (build-system gnu-build-system)
+    (inputs `(("munge" ,munge)
+              ("boost" ,boost)
+              ("libelf" ,libelf)
+              ("openmpi" ,openmpi)
+              ("libgcrypt" ,libgcrypt)
+              ("libgpg-error" ,libgpg-error)))
+    (synopsis "Software infrastructure for HPC environments")
+    (description
+     "LaunchMON is a software infrastructure that enables HPC
+(High-Performance Computing) run-time tools to co-locate tool daemons with a
+parallel job.  Its API allows a tool to identify all the remote processes of a
+job and to scalably launch daemons into the relevant nodes.")
+    (home-page "https://github.com/LLNL/LaunchMON")
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-10 10:03     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
@ 2018-04-10 10:03       ` Rouby Pierre-Antoine
  0 siblings, 0 replies; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-10 10:03 UTC (permalink / raw)
  To: 31080; +Cc: Rouby Pierre-Antoine

* gnu/packages/admin.scm (spindle): New variable.
---
 gnu/packages/admin.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index bf50a51f4..512dcf59c 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -2538,3 +2538,32 @@ parallel job.  Its API allows a tool to identify all the remote processes of a
 job and to scalably launch daemons into the relevant nodes.")
     (home-page "https://github.com/LLNL/LaunchMON")
     (license license:lgpl2.1)))
+
+(define-public spindle
+  (package
+    (name "spindle")
+    (version "0.10")
+    (source (origin
+              (file-name (git-file-name name version))
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/hpc/Spindle.git")
+                    (commit "ff922c1df167c28ab312d02217ed7be65ec0e831")))
+              (sha256
+               (base32
+                "15n3ay0qq81r5v7fif61q1vdjcq44pp2nynkh3fvbzc9fj3c39wd"))))
+    (build-system gnu-build-system)
+    (arguments '(#:configure-flags '("--enable-sec-launchmon"
+                                     "--enable-sec-munge"
+                                     "--enable-sec-none")))
+    (inputs `(("munge" ,munge)
+              ("openmpi" ,openmpi)
+              ("launchmon" ,launchmon)
+              ("libgcrypt" ,libgcrypt)))
+    (synopsis "Scalable dynamic library and Python loading in HPC
+environments")
+    (description
+     "Spindle is a tool for improving the performance of dynamic library and
+Python loading in HPC environments.")
+    (home-page "https://github.com/hpc/Spindle")
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 0/2] Add launchmon and spindle
  2018-04-09 13:50 ` [bug#31080] [PATCH 0/2] Add launchmon and spindle Ludovic Courtès
@ 2018-04-10 15:20   ` Eric Bavier
  2018-04-11 12:58     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
  0 siblings, 1 reply; 18+ messages in thread
From: Eric Bavier @ 2018-04-10 15:20 UTC (permalink / raw)
  To: Rouby Pierre-Antoine; +Cc: Ludovic Courtès, 31080

On Mon, Apr 09, 2018 at 03:50:34PM +0200, Ludovic Courtès wrote:
> I should say that this is Pierre-Antoine’s first week as an intern here
> at Inria to work on Guix for HPC.  So welcome, Pierre-Antoine!  :-)

Exciting, welcome Pierre-Antoine!  It will be nice to have another
person on Guix for HPC.

> The patches look good to me, but as Pierre-Antoine wrote, we’d like to
> get feedback from someone more familiar with the tools.  Eric maybe?

I have not used these tools much before, but I'll see if I can give
them a test run.  It will be a good excuse for me to finally get Guix
installed on one of our x86 systems. :)

Just a few comments/nitpicks on the patches:

> +
> +(define-public launchmon
> +  (package
> +    (name "launchmon")
> +    (version "1.0.2")
> +    (source (origin
> +              (file-name (git-file-name name version))
                  ^
Leave this field out, since the tarball file-name is already fine.

> +              (method url-fetch)
> +              (uri (string-append
> +                    "https://github.com/LLNL/LaunchMON/releases/download/v"
> +                    version "/" name "-v" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "0fm3nd9mydm9v2bf7bh01dbgrfnpwkapxa3dsvy3x1z0rz61qc0x"))))
> +    (build-system gnu-build-system)
> +    (inputs `(("munge" ,munge)
               ^
Please indent the inputs list like the other packages in this file.

> +              ("boost" ,boost)
> +              ("libelf" ,libelf)
> +              ("openmpi" ,openmpi)

IDK if this would be meaningful for these tools, but some other MPI
packages have used the convention of of naming this input just "mpi".
IIRC Ludovic suggested this in a paper.

> +              ("libgcrypt" ,libgcrypt)
> +              ("libgpg-error" ,libgpg-error)))
> +    (synopsis "Software infrastructure for HPC environments")

Maybe: "Launch tools into HPC environments" so as not to confuse it
with a general piece of "software infrastructure"

> +    (description
> +     "LaunchMON is a software infrastructure that enables HPC
> +(High-Performance Computing) run-time tools to co-locate tool daemons with a
> +parallel job.  Its API allows a tool to identify all the remote processes of a
> +job and to scalably launch daemons into the relevant nodes.")
> +    (home-page "https://github.com/LLNL/LaunchMON")
> +    (license license:lgpl2.1)))

We should also include a "supported-systems" field here:

   (supported-systems '("i686-linux" "x86_64-linux"))

Since only x86 and powerpc architectures are supported, as noted
above. :(

+
+(define-public spindle
+  (package
+    (name "spindle")
+    (version "0.10")
+    (source (origin
+              (file-name (git-file-name name version))
+              (method git-fetch)

Maybe just note in a comment that we're using a git checkout to avoid
auto-generated github tarballs.

+              (uri (git-reference
+                    (url "https://github.com/hpc/Spindle.git")
+                    (commit "ff922c1df167c28ab312d02217ed7be65ec0e831")))
                              ^
Could we reference the 'v0.10' tag instead?

+              (sha256
+               (base32
+                "15n3ay0qq81r5v7fif61q1vdjcq44pp2nynkh3fvbzc9fj3c39wd"))))
+    (build-system gnu-build-system)
+    (arguments '(#:configure-flags '("--enable-sec-launchmon"
+                                     "--enable-sec-munge"
+                                     "--enable-sec-none")))
+    (inputs `(("munge" ,munge)
             ^
nit: indentation

+              ("openmpi" ,openmpi)
+              ("launchmon" ,launchmon)
+              ("libgcrypt" ,libgcrypt)))
+    (synopsis "Scalable dynamic library and Python loading in HPC
+environments")

Maybe: "Scalable library loading in HPC environments"

+    (description
+     "Spindle is a tool for improving the performance of dynamic library and
+Python loading in HPC environments.")
+    (home-page "https://github.com/hpc/Spindle")
+    (license license:lgpl2.1)))


Otherwise LGTM.  I'll see if I can give them a try soon.

-- 
Eric Bavier, Scientific Libraries, Cray Inc.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 1/2] gnu: Add launchmon.
  2018-04-10 15:20   ` Eric Bavier
@ 2018-04-11 12:58     ` Rouby Pierre-Antoine
  2018-04-11 12:58       ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
  2018-04-11 15:28       ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Eric Bavier
  0 siblings, 2 replies; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-11 12:58 UTC (permalink / raw)
  To: 31080; +Cc: bavier, ludovic.courtes, Rouby Pierre-Antoine

* gnu/packages/admin.scm (launchmon): New variable.
---
 gnu/packages/admin.scm | 36 +++++++++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 6a7bed389..71899d98c 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -20,6 +20,7 @@
 ;;; Copyright © 2017 Christopher Allan Webber <cwebber@dustycloud.org>
 ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
 ;;; Copyright © 2018 Arun Isaac <arunisaac@systemreboot.net>
+;;; Copyright © 2018 Pierre-Antoine Rouby <pierre-antoine.rouby@inria.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -90,7 +91,10 @@
   #:use-module (gnu packages gnome)
   #:use-module (gnu packages kerberos)
   #:use-module (gnu packages gtk)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages boost)
+  #:use-module (gnu packages elf)
+  #:use-module (gnu packages mpi))
 
 (define-public aide
   (package
@@ -2505,3 +2509,33 @@ printed instead of after the entire file has been read, which is often too
 late.")
     (home-page "https://jwilk.net/software/hungrycat")
     (license license:expat)))
+
+(define-public launchmon
+  (package
+    (name "launchmon")
+    (version "1.0.2")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "https://github.com/LLNL/LaunchMON/releases/download/v"
+                    version "/" name "-v" version ".tar.gz"))
+              (sha256
+               (base32
+                "0fm3nd9mydm9v2bf7bh01dbgrfnpwkapxa3dsvy3x1z0rz61qc0x"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("mpi" ,openmpi)
+       ("munge" ,munge)
+       ("boost" ,boost)
+       ("libelf" ,libelf)
+       ("libgcrypt" ,libgcrypt)
+       ("libgpg-error" ,libgpg-error)))
+    (synopsis "An infrastructue for large scale tool daemon launching")
+    (description
+     "LaunchMON is a software infrastructure that enables HPC run-time
+tools to co-locate tool daemons with a parallel job.  Its API allows a
+tool to identify all the remote processes of a job and to scalably
+launch daemons into the relevant nodes.")
+    (home-page "https://github.com/LLNL/LaunchMON")
+    (supported-systems '("i686-linux" "x86_64-linux"))
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-11 12:58     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
@ 2018-04-11 12:58       ` Rouby Pierre-Antoine
  2018-04-11 15:31         ` Eric Bavier
  2018-04-11 15:28       ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Eric Bavier
  1 sibling, 1 reply; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-11 12:58 UTC (permalink / raw)
  To: 31080; +Cc: bavier, ludovic.courtes, Rouby Pierre-Antoine

* gnu/packages/admin.scm (spindle): New variable.
---
 gnu/packages/admin.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 71899d98c..a277a0a75 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -2539,3 +2539,32 @@ launch daemons into the relevant nodes.")
     (home-page "https://github.com/LLNL/LaunchMON")
     (supported-systems '("i686-linux" "x86_64-linux"))
     (license license:lgpl2.1)))
+
+(define-public spindle
+  (package
+    (name "spindle")
+    (version "0.10")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "https://github.com/hpc/Spindle/archive/v"
+                    version ".tar.gz"))
+              (file-name (string-append name "-" version ".tar.gz"))
+              (sha256
+               (base32
+                "10hl1d5nvjbx8yah4zhbxhijjigqsvrld057ljv01cvqkqycsx39"))))
+    (build-system gnu-build-system)
+    (arguments '(#:configure-flags '("--enable-sec-launchmon"
+                                     "--enable-sec-munge"
+                                     "--enable-sec-none")))
+    (inputs
+     `(("mpi" ,openmpi)
+       ("munge" ,munge)
+       ("launchmon" ,launchmon)
+       ("libgcrypt" ,libgcrypt)))
+    (synopsis "Scalable library loading in HPC environments")
+    (description
+     "Spindle is a tool for improving the performance of dynamic library and
+Python loading in HPC environments.")
+    (home-page "https://github.com/hpc/Spindle")
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 1/2] gnu: Add launchmon.
  2018-04-11 12:58     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
  2018-04-11 12:58       ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
@ 2018-04-11 15:28       ` Eric Bavier
  1 sibling, 0 replies; 18+ messages in thread
From: Eric Bavier @ 2018-04-11 15:28 UTC (permalink / raw)
  To: Rouby Pierre-Antoine; +Cc: ludovic.courtes, 31080

On Wed, Apr 11, 2018 at 02:58:09PM +0200, Rouby Pierre-Antoine wrote:
> @@ -2505,3 +2509,33 @@ printed instead of after the entire file has been read, which is often too
>  late.")
>      (home-page "https://jwilk.net/software/hungrycat")
>      (license license:expat)))
> +
> +(define-public launchmon
> +  (package
> +    (name "launchmon")
> +    (version "1.0.2")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                    "https://github.com/LLNL/LaunchMON/releases/download/v"
> +                    version "/" name "-v" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "0fm3nd9mydm9v2bf7bh01dbgrfnpwkapxa3dsvy3x1z0rz61qc0x"))))
> +    (build-system gnu-build-system)
> +    (inputs
> +     `(("mpi" ,openmpi)
> +       ("munge" ,munge)
> +       ("boost" ,boost)
> +       ("libelf" ,libelf)
> +       ("libgcrypt" ,libgcrypt)
> +       ("libgpg-error" ,libgpg-error)))
> +    (synopsis "An infrastructue for large scale tool daemon launching")
                  ^
No "An" at the beginning.  See the "Synopses and Descrtiptions"
section in the manual.

Otherwise LGTM!  Thanks!

-- 
Eric Bavier, Scientific Libraries, Cray Inc.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-11 12:58       ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
@ 2018-04-11 15:31         ` Eric Bavier
  2018-04-12  8:43           ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
  0 siblings, 1 reply; 18+ messages in thread
From: Eric Bavier @ 2018-04-11 15:31 UTC (permalink / raw)
  To: Rouby Pierre-Antoine; +Cc: ludovic.courtes, 31080

On Wed, Apr 11, 2018 at 02:58:10PM +0200, Rouby Pierre-Antoine wrote:
> +(define-public spindle
> +  (package
> +    (name "spindle")
> +    (version "0.10")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                    "https://github.com/hpc/Spindle/archive/v"
> +                    version ".tar.gz"))

No no, the git checkout used before was good.  These autogenerated
tarballs (as opposed to e.g. the maintainer-uploaded tarball that
launchmon has) can change at any time, so we'd rather not use them.
Sorry for any confusion my comment caused.

Otherwise looks good.

-- 
Eric Bavier, Scientific Libraries, Cray Inc.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 1/2] gnu: Add launchmon.
  2018-04-11 15:31         ` Eric Bavier
@ 2018-04-12  8:43           ` Rouby Pierre-Antoine
  2018-04-12  8:43             ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
  0 siblings, 1 reply; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-12  8:43 UTC (permalink / raw)
  To: 31080; +Cc: bavier, ludovic.courtes, Rouby Pierre-Antoine

* gnu/packages/admin.scm (launchmon): New variable.
---
 gnu/packages/admin.scm | 36 +++++++++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 6a7bed389..c9fea40b1 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -20,6 +20,7 @@
 ;;; Copyright © 2017 Christopher Allan Webber <cwebber@dustycloud.org>
 ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
 ;;; Copyright © 2018 Arun Isaac <arunisaac@systemreboot.net>
+;;; Copyright © 2018 Pierre-Antoine Rouby <pierre-antoine.rouby@inria.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -90,7 +91,10 @@
   #:use-module (gnu packages gnome)
   #:use-module (gnu packages kerberos)
   #:use-module (gnu packages gtk)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages boost)
+  #:use-module (gnu packages elf)
+  #:use-module (gnu packages mpi))
 
 (define-public aide
   (package
@@ -2505,3 +2509,33 @@ printed instead of after the entire file has been read, which is often too
 late.")
     (home-page "https://jwilk.net/software/hungrycat")
     (license license:expat)))
+
+(define-public launchmon
+  (package
+    (name "launchmon")
+    (version "1.0.2")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "https://github.com/LLNL/LaunchMON/releases/download/v"
+                    version "/" name "-v" version ".tar.gz"))
+              (sha256
+               (base32
+                "0fm3nd9mydm9v2bf7bh01dbgrfnpwkapxa3dsvy3x1z0rz61qc0x"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("mpi" ,openmpi)
+       ("munge" ,munge)
+       ("boost" ,boost)
+       ("libelf" ,libelf)
+       ("libgcrypt" ,libgcrypt)
+       ("libgpg-error" ,libgpg-error)))
+    (synopsis "Infrastructue for large scale tool daemon launching")
+    (description
+     "LaunchMON is a software infrastructure that enables HPC run-time
+tools to co-locate tool daemons with a parallel job.  Its API allows a
+tool to identify all the remote processes of a job and to scalably
+launch daemons into the relevant nodes.")
+    (home-page "https://github.com/LLNL/LaunchMON")
+    (supported-systems '("i686-linux" "x86_64-linux"))
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-12  8:43           ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
@ 2018-04-12  8:43             ` Rouby Pierre-Antoine
  2018-04-18 15:07               ` Eric Bavier
  0 siblings, 1 reply; 18+ messages in thread
From: Rouby Pierre-Antoine @ 2018-04-12  8:43 UTC (permalink / raw)
  To: 31080; +Cc: bavier, ludovic.courtes, Rouby Pierre-Antoine

* gnu/packages/admin.scm (spindle): New variable.
---
 gnu/packages/admin.scm | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index c9fea40b1..8ea46b7e2 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -2539,3 +2539,33 @@ launch daemons into the relevant nodes.")
     (home-page "https://github.com/LLNL/LaunchMON")
     (supported-systems '("i686-linux" "x86_64-linux"))
     (license license:lgpl2.1)))
+
+(define-public spindle
+  (package
+    (name "spindle")
+    (version "0.10")
+    (source (origin
+              ;; We use git checkout to avoid github auto-generated tarballs
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/hpc/Spindle.git")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "15n3ay0qq81r5v7fif61q1vdjcq44pp2nynkh3fvbzc9fj3c39wd"))))
+    (build-system gnu-build-system)
+    (arguments '(#:configure-flags '("--enable-sec-launchmon"
+                                     "--enable-sec-munge"
+                                     "--enable-sec-none")))
+    (inputs
+     `(("mpi" ,openmpi)
+       ("munge" ,munge)
+       ("launchmon" ,launchmon)
+       ("libgcrypt" ,libgcrypt)))
+    (synopsis "Scalable library loading in HPC environments")
+    (description
+     "Spindle is a tool for improving the performance of dynamic library and
+Python loading in HPC environments.")
+    (home-page "https://github.com/hpc/Spindle")
+    (license license:lgpl2.1)))
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-12  8:43             ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
@ 2018-04-18 15:07               ` Eric Bavier
  2018-04-18 15:21                 ` Pierre-Antoine Rouby
  2018-04-19  9:22                 ` bug#31080: " Ludovic Courtès
  0 siblings, 2 replies; 18+ messages in thread
From: Eric Bavier @ 2018-04-18 15:07 UTC (permalink / raw)
  To: Rouby Pierre-Antoine; +Cc: ludovic.courtes, 31080

On Thu, Apr 12, 2018 at 10:43:11AM +0200, Rouby Pierre-Antoine wrote:
> * gnu/packages/admin.scm (spindle): New variable.
> ---
>  gnu/packages/admin.scm | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 

These look good to me now.  IMO they can be pushed now, and we can fix
any bugs with actual usage once those tests are done.

-- 
Eric Bavier, Scientific Libraries, Cray Inc.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [bug#31080] [PATCH 2/2] gnu: Add spindle.
  2018-04-18 15:07               ` Eric Bavier
@ 2018-04-18 15:21                 ` Pierre-Antoine Rouby
  2018-04-19  9:22                 ` bug#31080: " Ludovic Courtès
  1 sibling, 0 replies; 18+ messages in thread
From: Pierre-Antoine Rouby @ 2018-04-18 15:21 UTC (permalink / raw)
  To: 31080; +Cc: Eric Bavier

> These look good to me now.  IMO they can be pushed now, and we can fix
> any bugs with actual usage once those tests are done.

Ok, thanks you for reviewing my code.

Pierre-Antoine Rouby

^ permalink raw reply	[flat|nested] 18+ messages in thread

* bug#31080: [PATCH 2/2] gnu: Add spindle.
  2018-04-18 15:07               ` Eric Bavier
  2018-04-18 15:21                 ` Pierre-Antoine Rouby
@ 2018-04-19  9:22                 ` Ludovic Courtès
  1 sibling, 0 replies; 18+ messages in thread
From: Ludovic Courtès @ 2018-04-19  9:22 UTC (permalink / raw)
  To: Eric Bavier; +Cc: 31080-done, Rouby Pierre-Antoine

Hello,

Eric Bavier <bavier@cray.com> skribis:

> On Thu, Apr 12, 2018 at 10:43:11AM +0200, Rouby Pierre-Antoine wrote:
>> * gnu/packages/admin.scm (spindle): New variable.
>> ---
>>  gnu/packages/admin.scm | 30 ++++++++++++++++++++++++++++++
>>  1 file changed, 30 insertions(+)
>> 
>
> These look good to me now.  IMO they can be pushed now, and we can fix
> any bugs with actual usage once those tests are done.

Alright, applied.  Thanks for reviewing Eric, and thanks for these first
patches P-A!

Ludo’.

^ permalink raw reply	[flat|nested] 18+ messages in thread

end of thread, other threads:[~2018-04-19  9:24 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-06 16:02 [bug#31080] [PATCH 0/2] Add launchmon and spindle Rouby Pierre-Antoine
2018-04-09  6:58 ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
2018-04-09  6:59 ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
2018-04-09 14:06   ` Jonathan Brielmaier
2018-04-09 20:34     ` Ludovic Courtès
2018-04-10 10:03     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
2018-04-10 10:03       ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
2018-04-09 13:50 ` [bug#31080] [PATCH 0/2] Add launchmon and spindle Ludovic Courtès
2018-04-10 15:20   ` Eric Bavier
2018-04-11 12:58     ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
2018-04-11 12:58       ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
2018-04-11 15:31         ` Eric Bavier
2018-04-12  8:43           ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Rouby Pierre-Antoine
2018-04-12  8:43             ` [bug#31080] [PATCH 2/2] gnu: Add spindle Rouby Pierre-Antoine
2018-04-18 15:07               ` Eric Bavier
2018-04-18 15:21                 ` Pierre-Antoine Rouby
2018-04-19  9:22                 ` bug#31080: " Ludovic Courtès
2018-04-11 15:28       ` [bug#31080] [PATCH 1/2] gnu: Add launchmon Eric Bavier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).