Skip to content

Update file permissions in test_cufile to use 600 instead of 644 - #910

Merged
leofang merged 1 commit into
NVIDIA:mainfrom
kkraus14:cufile_test_permissions
Aug 27, 2025
Merged

Update file permissions in test_cufile to use 600 instead of 644#910
leofang merged 1 commit into
NVIDIA:mainfrom
kkraus14:cufile_test_permissions

Conversation

@kkraus14

Copy link
Copy Markdown
Collaborator

Description

Changes the permissions of the files from 644 to 600 to appease CodeQL. There was no security risk as this is only testing code, but may as well limit to the minimal permissions needed.

@copy-pr-bot

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@kkraus14

Copy link
Copy Markdown
CollaboratorAuthor

/ok to test

@kkraus14kkraus14 added the to-be-backported Trigger the bot to raise a backport PR upon merge label Aug 27, 2025
@github-actions

This comment has been minimized.

@leofangleofang added P1 Medium priority - Should do test Improvements or additions to tests cuda.bindings Everything related to the cuda.bindings module labels Aug 27, 2025
@leofang
leofang merged commit 2bd44a2 into NVIDIA:mainAug 27, 2025
52 checks passed
github-actionsBot pushed a commit that referenced this pull request Aug 27, 2025
@github-actions

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

leofang pushed a commit that referenced this pull request Aug 27, 2025
)
(cherry picked from commit 2bd44a2)
Co-authored-by: Keith Kraus <keith.j.kraus@gmail.com>
@cpcloudcpcloud mentioned this pull request Oct 8, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindingsEverything related to the cuda.bindings moduleP1Medium priority - Should dotestImprovements or additions to teststo-be-backportedTrigger the bot to raise a backport PR upon merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@kkraus14@leofang