fix bug in validation logic (#3915)

* fix bug in validation logic * test --------- Co-authored-by: Richard Kuo (Danswer) <rkuo@onyx.app>
onyx-dot-app · Feb 5, 2025 · 47e6192 · 47e6192
1 parent 29f5f4e
commit 47e6192
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/backend/onyx/background/celery/tasks/doc_permission_syncing/tasks.py b/backend/onyx/background/celery/tasks/doc_permission_syncing/tasks.py
@@ -659,7 +659,8 @@ def validate_permission_sync_fence(
         f"tasks_scanned={tasks_scanned} tasks_not_in_celery={tasks_not_in_celery}"
     )
 
-    if tasks_not_in_celery == 0:
+    # we're only active if tasks_scanned > 0 and tasks_not_in_celery == 0
+    if tasks_scanned > 0 and tasks_not_in_celery == 0:
         redis_connector.permissions.set_active()
         return