Skip to content

ROX-33222: handle multiple hardlinks - #1468

Draft
ovalenti wants to merge 3 commits into
mainfrom
ROX-33222-multiple-hardlinks
Draft

ROX-33222: handle multiple hardlinks#1468
ovalenti wants to merge 3 commits into
mainfrom
ROX-33222-multiple-hardlinks

Conversation

@ovalenti

Copy link
Copy Markdown

Description

When files are tracked via their inode (host), we need to take into account that access may be done using a dentry (path) that is not the monitored one. Also, we need to adapt the kernel "monitored" inode life-cycle to verify if there are paths still monitored. The host_scanner model has to be updated to support several paths per inode.

Checklist

  • Patch has a change log entry OR does not need one.
  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@ovalenti
ovalenti force-pushed the ROX-33222-multiple-hardlinks branch from fc5e66b to 9fb28c1 Compare August 13, 2026 22:29
@codecov-commenter

codecov-commenter commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.96%. Comparing base (d1ad908) to head (e92a6f1).
⚠️ Report is 42 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1468      +/-   ##
==========================================
- Coverage   34.52%   33.96%   -0.56%     
==========================================
  Files          22       22              
  Lines        3325     3421      +96     
  Branches     3325     3421      +96     
==========================================
+ Hits         1148     1162      +14     
- Misses       2172     2254      +82     
  Partials        5        5              

☔ View full report in Codecov by Harness.
📢 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.

@ovalenti
ovalenti force-pushed the ROX-33222-multiple-hardlinks branch from 9a2b772 to e92a6f1 Compare August 27, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants