Skip to content

[2.10] [MOD-10076] - remove the FDs check in gc oom test#6926

Merged
redisearch-backport-pull-request[bot] merged 1 commit into2.10from
backport-6903-to-2.10
Sep 29, 2025
Merged

[2.10] [MOD-10076] - remove the FDs check in gc oom test#6926
redisearch-backport-pull-request[bot] merged 1 commit into2.10from
backport-6903-to-2.10

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


Note

Removes psutil-based open-file descriptor assertions from test_gc_oom and adjusts GC invocation to run once after deletions.

  • Tests:
    • tests/pytests/test_gc.py:
      • Remove psutil import, get_open_file_count, and all open-FD assertions from test_gc_oom.
      • Move forceInvokeGC(env) to run once after deleting all docs; keep GC bytes_collected assertions unchanged.

Written by Cursor Bugbot for commit d24f633. 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)
@codecov
Copy link

codecov bot commented Sep 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.29%. Comparing base (d5c85db) to head (d24f633).
⚠️ Report is 1 commits behind head on 2.10.

Additional details and impacted files
@@            Coverage Diff             @@
##             2.10    #6926      +/-   ##
==========================================
- Coverage   89.32%   89.29%   -0.03%     
==========================================
  Files         207      207              
  Lines       36165    36165              
==========================================
- Hits        32304    32295       -9     
- Misses       3861     3870       +9     
Flag Coverage Δ
flow 83.81% <ø> (-0.15%) ⬇️
unit 42.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@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 2.10 with commit 6f949eb Sep 29, 2025
16 checks passed
@redisearch-backport-pull-request redisearch-backport-pull-request bot deleted the backport-6903-to-2.10 branch September 29, 2025 11:39
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