[ARVADOS] updated: 2.1.0-1819-g5e06ca0b4

[Git user]

Summary of changes:
 lib/crunchrun/cuda.go | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

       via  5e06ca0b451f36be33396f8e83bdaa4f9d6f74bb (commit)
      from  43be77c2f1b4b972113202bcd3d543fe0428778a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 5e06ca0b451f36be33396f8e83bdaa4f9d6f74bb
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Mon Jan 10 15:39:26 2022 -0500

    12630: Tweak nvidia warnings
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/lib/crunchrun/cuda.go b/lib/crunchrun/cuda.go
index 8282359ea..6efc689ba 100644
--- a/lib/crunchrun/cuda.go
+++ b/lib/crunchrun/cuda.go
@@ -5,15 +5,13 @@
 package crunchrun
 
 import (
-	"fmt"
-	"io"
 	"os/exec"
 )
 
 // nvidiaModprobe makes sure all the nvidia kernel modules and devices
 // are set up.  If we don't have all the modules/devices set up we get
 // "CUDA_ERROR_UNKNOWN".
-func nvidiaModprobe(writer io.Writer) {
+func nvidiaModprobe(writer *ThrottledLogger) {
 	// The underlying problem is that when normally running
 	// directly on the host, the CUDA SDK will automatically
 	// detect and set up the devices on demand.  However, when
@@ -44,7 +42,7 @@ func nvidiaModprobe(writer io.Writer) {
 	nvidiaSmi.Stderr = writer
 	err := nvidiaSmi.Run()
 	if err != nil {
-		writer.Write([]byte(fmt.Sprintf("nvidia-smi error: %v\n", err)))
+		writer.Printf("Warning - nvidia-smi: %v", err)
 	}
 
 	// Load the kernel modules & devices associated with
@@ -65,7 +63,7 @@ func nvidiaModprobe(writer io.Writer) {
 		nvmodprobe.Stderr = writer
 		err = nvmodprobe.Run()
 		if err != nil {
-			writer.Write([]byte(fmt.Sprintf("nvidia-modprobe error: %v\n", err)))
+			writer.Printf("Warning: nvidia-modprobe: %v", err)
 		}
 	}
 }

-----------------------------------------------------------------------


hooks/post-receive
--