fix

edly-io · May 30, 2024 · 6478189 · 6478189
1 parent af7fb34
commit 6478189
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/course_discovery/apps/edx_haystack_extensions/management/commands/update_index.py b/course_discovery/apps/edx_haystack_extensions/management/commands/update_index.py
@@ -56,10 +56,12 @@ def handle(self, **options):
                     record_count_is_sane, index_info_string = self.sanity_check_new_index(
                         backend.conn, index, record_count
                     )
-                    if record_count_is_sane:
-                        indexes_pending.pop(index, None)
-                    else:
+                    if not record_count_is_sane:
                         indexes_pending[index] = index_info_string
+                    else:
+                        indexes_pending.pop(index, None)
+                else:
+                    indexes_pending.pop(index, None)
 
         if indexes_pending:
             raise CommandError(f'Sanity check failed for new index(es): {indexes_pending}')