From f61a39eb70046280bd4611ea2dad38a5602303dc Mon Sep 17 00:00:00 2001
From: Nico Cesar <nico@curoverse.com>
Date: Fri, 21 Oct 2016 07:38:37 -0400
Subject: [PATCH] 10232: dont create user logic was wrong

refs #10232
---
 services/login-sync/bin/arvados-login-sync | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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.
-- 
2.30.2