arvados-login-sync 2.3.3 → 2.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ebbd1632924ac9c57a7c27c1add22dea0c0b1238b897ba95eec585ebe5f07572
-  data.tar.gz: d833cf1bb9cd391171af4fb6d03de128f50382f85133ee94687d8be33dbe961e
+  metadata.gz: 187519e067d9a0a5019d04997010cbeb0b542f736c5c2585fbc0cd8d0d141cc1
+  data.tar.gz: 18e7c489bb7b394529211f395cd7c52706fccc104f9bbad9a4fe30c31923a0ab
 SHA512:
-  metadata.gz: 9e05c28aa92f5eb6a3de3f4ae4234c20d98bf0ae77bd6ee977b8207c20d7d7303705e30220e9cc8de22e1cf3d34b63d42226629cbd7ecbef928759b105a0f7fe
-  data.tar.gz: 03a8045c5754ccf41cae31daf89aa1c0da3f8d74079cb46a759ee035d6fda608a7b2d230c4064384e0339b4d0bdfdcacd7b61e08ff21d457fdc4dbf26c9dfa09
+  metadata.gz: fa346fc2bc893d8ee22f52a1eb9a505ac99523e4de947d5cfbf69554812b22e40fcbcb2c2175912365428b3574a0fdba6220b23bf51b5023e5990a3fb71aa376
+  data.tar.gz: 637a95b6b14c4480e830bf160c66adf076cd0afa7363b3704aca33496c1a7696956ed517a4a0b2b284ae239aa1745fe2489a382fc0af914b61570fe399073f5e

data/bin/arvados-login-sync

@@ -10,6 +10,7 @@ require 'etc'
 require 'fileutils'
 require 'yaml'
 require 'optparse'
+require 'open3'
 
 req_envs = %w(ARVADOS_API_HOST ARVADOS_API_TOKEN ARVADOS_VIRTUAL_MACHINE_UUID)
 req_envs.each do |k|
@@ -124,11 +125,12 @@ begin
     unless pwnam[l[:username]]
       STDERR.puts "Creating account #{l[:username]}"
       # Create new user
-      unless system("useradd", "-m",
+      out, st = Open3.capture2e("useradd", "-m",
                 "-c", username,
                 "-s", "/bin/bash",
                 username)
-        STDERR.puts "Account creation failed for #{l[:username]}: #{$?}"
+      if st.exitstatus != 0
+        STDERR.puts "Account creation failed for #{l[:username]}:\n#{out}"
         next
       end
       begin
@@ -150,7 +152,10 @@ begin
       if existing_groups.index(addgroup).nil?
         # User should be in group, but isn't, so add them.
         STDERR.puts "Add user #{username} to #{addgroup} group"
-        system("usermod", "-aG", addgroup, username)
+        out, st = Open3.capture2e("usermod", "-aG", addgroup, username)
+        if st.exitstatus != 0
+          STDERR.puts "Failed to add #{username} to #{addgroup} group:\n#{out}"
+        end
       end
     end
 
@@ -158,7 +163,10 @@ begin
       if groups.index(removegroup).nil?
         # User is in a group, but shouldn't be, so remove them.
         STDERR.puts "Remove user #{username} from #{removegroup} group"
-        system("gpasswd", "-d", username, removegroup)
+        out, st = Open3.capture2e("gpasswd", "-d", username, removegroup)
+        if st.exitstatus != 0
+          STDERR.puts "Failed to remove user #{username} from #{removegroup} group:\n#{out}"
+        end
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: arvados-login-sync
 version: !ruby/object:Gem::Version
-  version: 2.3.3
+  version: 2.4.2
 platform: ruby
 authors:
 - Arvados Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-11-17 00:00:00.000000000 Z
+date: 2022-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: arvados
@@ -73,7 +73,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.12'
 description: Creates and updates local login accounts for Arvados users. Built from
-  git commit bc9d8d1e4caeef8c4b2da02f9a134fc7b57148d7
+  git commit b8c1729e63dcf8e94b296d6ee90cf7c16c79d72c
 email: packaging@arvados.org
 executables:
 - arvados-login-sync