arvados-login-sync 2.4.1 → 2.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 458625df9fb5c04731f2fbb238f53c985fa6942692792663a0a40f010ffba380
-  data.tar.gz: a772e4b01cc40f57b8b89d3c060aefac6ae1d72db5d79793032aebd9da195338
+  metadata.gz: 187519e067d9a0a5019d04997010cbeb0b542f736c5c2585fbc0cd8d0d141cc1
+  data.tar.gz: 18e7c489bb7b394529211f395cd7c52706fccc104f9bbad9a4fe30c31923a0ab
 SHA512:
-  metadata.gz: 4f3bb203254f4c166bb2a525af3893cb40ee5ba2a93c167f1011ac30c3e54b40a8197ffd279732c249d5b7ccb3b082f9765302a763ad2deb48a50e1bd4131491
-  data.tar.gz: 39d1525c7dfbbede9e0002540ce370fbb057c39cdf61516233d7ee913e51ef590eff87463002799bbdfe1dbdf7e82ccbbcf93b413c3c11d4f4fa55e94231fc12
+  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.4.1
+  version: 2.4.2
 platform: ruby
 authors:
 - Arvados Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-10-29 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 0c9dda66d44f466c6ef4b2c0a8b3c4e685b74143
+  git commit b8c1729e63dcf8e94b296d6ee90cf7c16c79d72c
 email: packaging@arvados.org
 executables:
 - arvados-login-sync