Skip to content

[release-1.19] fix: Additional GIDs are dropped due to file mode mask#1752

Open
github-actions[bot] wants to merge 1 commit intorelease-1.19from
backport-1745-to-release-1.19
Open

[release-1.19] fix: Additional GIDs are dropped due to file mode mask#1752
github-actions[bot] wants to merge 1 commit intorelease-1.19from
backport-1745-to-release-1.19

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions bot commented Apr 6, 2026

🤖 Automated backport of #1745 to release-1.19

✅ Cherry-pick completed successfully with no conflicts.

Original PR: #1745
Original Author: @elezar

Cherry-picked commits (1):

  • 2e9a666 fix: Additional GIDs are dropped due to file mode mask

This backport was automatically created by the backport bot.

The filemode returned from DeviceFromPath clears the bits that
are track whether the device is a character device. This means
that the check that ensures that the mode includes the character
device bits always fails and no additional GIDs are detected.

This change removes the check. In practice this should have no
effect since we only ever detect char devices.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
Co-authored-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
(cherry picked from commit 2e9a666)
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Apr 6, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@cdesiniotis
Copy link
Copy Markdown
Contributor

/ok to test 3cf726c

@coveralls
Copy link
Copy Markdown

Coverage Report for PR #1752

Coverage increased (+0.06%) to 43.406%

Diff Coverage: No coverable lines changed

No coverage regressions found.


Coverage Status
Change from base Build 22981003708: 0.06%
Covered Lines: 6434
Relevant Lines: 14823

💛 - Coveralls

@cdesiniotis cdesiniotis enabled auto-merge April 6, 2026 19:01
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.

3 participants