10232: dont create user logic was wrong
authorNico Cesar <nico@curoverse.com>
Fri, 21 Oct 2016 11:38:37 +0000 (07:38 -0400)
committerNico Cesar <nico@curoverse.com>
Fri, 21 Oct 2016 11:38:37 +0000 (07:38 -0400)
refs #10232

services/login-sync/bin/arvados-login-sync

index 46e85bc802992085b306d8512d4da05a15b11268..57487711d5235ccca9dd5f55c6872a5a629584a2 100755 (executable)
@@ -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.