[arvados] updated: 2.6.0-273-ge6276d3b3

[git repository hosting]

Summary of changes:
 services/login-sync/bin/arvados-login-sync | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

       via  e6276d3b309b3710a92adb308f879eff7db44488 (commit)
      from  cc0c17337c94152134d33f2a9d5a91389508307c (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 e6276d3b309b3710a92adb308f879eff7db44488
Author: Lucas Di Pentima <lucas.dipentima at curii.com>
Date:   Tue Jun 20 11:44:08 2023 -0300

    20643: Skips user account processing earlier to avoid any partial configs.
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima at curii.com>

diff --git a/services/login-sync/bin/arvados-login-sync b/services/login-sync/bin/arvados-login-sync
index 0a9648db9..9bcb5bfa9 100755
--- a/services/login-sync/bin/arvados-login-sync
+++ b/services/login-sync/bin/arvados-login-sync
@@ -146,6 +146,12 @@ begin
       end
     end
 
+    homedir = pwnam[l[:username]].dir
+    if !File.exist?(homedir)
+      STDERR.puts "Cannot set up user #{username} because their home directory #{homedir} does not exist. Skipping."
+      next
+    end
+
     existing_groups = current_user_groups[username] || []
     groups = l[:groups] || []
     # Adding users to the FUSE group has long been hardcoded behavior.
@@ -175,11 +181,6 @@ begin
       end
     end
 
-    homedir = pwnam[l[:username]].dir
-    if !File.exist?(homedir)
-      STDERR.puts "Cannot set up user #{username} because their home directory #{homedir} does not exist. Skipping."
-      next
-    end
     userdotssh = File.join(homedir, ".ssh")
     Dir.mkdir(userdotssh) if !File.exist?(userdotssh)
 

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


hooks/post-receive
--