From: Nico Cesar <nico@curoverse.com>
Date: Fri, 21 Oct 2016 11:38:37 +0000 (-0400)
Subject: 10232: dont create user logic was wrong
X-Git-Tag: 1.1.0~639^2~1
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/f61a39eb70046280bd4611ea2dad38a5602303dc

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 46e85bc802..57487711d5 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.