[ARVADOS] updated: f61a39eb70046280bd4611ea2dad38a5602303dc

[Git user]

Summary of changes:
 services/login-sync/bin/arvados-login-sync | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  f61a39eb70046280bd4611ea2dad38a5602303dc (commit)
      from  b8a53fdbe406c1c46ba39facccec63c5055001bf (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 f61a39eb70046280bd4611ea2dad38a5602303dc
Author: Nico Cesar <nico at curoverse.com>
Date:   Fri Oct 21 07:38:37 2016 -0400

    10232: dont create user logic was wrong
    
    refs #10232

diff --git a/services/login-sync/bin/arvados-login-sync b/services/login-sync/bin/arvados-login-sync
index 46e85bc..5748771 100755
--- a/services/login-sync/bin/arvados-login-sync
+++ b/services/login-sync/bin/arvados-login-sync
@@ -80,7 +80,7 @@ begin
     next if seen[l[:username]]
     seen[l[:username]] = true if not seen.has_key?(l[:username])
 
-    unless uids[l[:username]] and not dont_create_user
+    unless uids[l[:username]] or dont_create_user
       STDERR.puts "Creating account #{l[:username]}"
       groups = l[:groups] || []
       # Adding users to the FUSE group has long been hardcoded behavior.

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


hooks/post-receive
--