Skip to content

[8.2] [MOD-10076] - remove the FDs check in gc oom test#6927

Merged
redisearch-backport-pull-request[bot] merged 1 commit into8.2from
backport-6903-to-8.2
Sep 29, 2025
Merged

[8.2] [MOD-10076] - remove the FDs check in gc oom test#6927
redisearch-backport-pull-request[bot] merged 1 commit into8.2from
backport-6903-to-8.2

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

@redisearch-backport-pull-request redisearch-backport-pull-request bot commented Sep 29, 2025

Description

Backport of #6903 to 8.2.


Note

Removes psutil-based file descriptor checks from test_gc_oom and runs GC once after deletions instead of per-document.

  • Tests:
    • tests/pytests/test_gc.py:
      • Remove psutil import/utility and all open file descriptor assertions from test_gc_oom.
      • Change GC invocation to a single forceInvokeGC(env) after deleting all docs.
      • Keep GC byte-collection assertions for OOM and post-oom scenarios.

Written by Cursor Bugbot for commit 7992250. This will update automatically on new commits. Configure here.

* add wait for gc

* remove FDs check

* add wait for gc

* remove FDs check

(cherry picked from commit 44d4914)
@redisearch-backport-pull-request redisearch-backport-pull-request bot added this pull request to the merge queue Sep 29, 2025
Merged via the queue into 8.2 with commit 8895096 Sep 29, 2025
29 checks passed
@redisearch-backport-pull-request redisearch-backport-pull-request bot deleted the backport-6903-to-8.2 branch September 29, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant