Skip to content

DAOS-16362 pydaos: remove execute perm bit on checkpoint files#17651

Merged
daltonbohning merged 3 commits intomasterfrom
0xe0f/fix-pytorch-checkpoint-file-permission
Apr 6, 2026
Merged

DAOS-16362 pydaos: remove execute perm bit on checkpoint files#17651
daltonbohning merged 3 commits intomasterfrom
0xe0f/fix-pytorch-checkpoint-file-permission

Conversation

@enakta
Copy link
Copy Markdown
Contributor

@enakta enakta commented Mar 6, 2026

There's no need to set execute permission bit for the checkpoint files.

Probably it was set by mistake.

Features: pytorch

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

There's no need to set execute permission bit for the checkpoint files.

Features: pytorch

Signed-off-by: Denis Barakhtanov <dbarahtanov@enakta.com>
@enakta enakta requested review from a team as code owners March 6, 2026 05:17
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 6, 2026

Ticket title is 'pytorch checkpoint module'
Status is 'Resolved'
https://daosio.atlassian.net/browse/DAOS-16362

@daosbuild3
Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17651/1/execution/node/1362/log

@daltonbohning
Copy link
Copy Markdown
Contributor

Test failure is existing and unrelated: https://daosio.atlassian.net/browse/DAOS-18649

@daltonbohning daltonbohning requested a review from a team April 6, 2026 22:45
@daltonbohning daltonbohning merged commit fe4ac69 into master Apr 6, 2026
40 of 41 checks passed
@daltonbohning daltonbohning deleted the 0xe0f/fix-pytorch-checkpoint-file-permission branch April 6, 2026 22:45
0xE0F pushed a commit that referenced this pull request Apr 6, 2026
There's no need to set execute permission bit for the checkpoint files.

Signed-off-by: Denis Barakhtanov <dbarahtanov@enakta.com>
@daltonbohning daltonbohning added waiting-for-merge-approval Waiting for merge approval and removed waiting-for-merge-approval Waiting for merge approval labels Apr 7, 2026
daltonbohning pushed a commit that referenced this pull request Apr 8, 2026
… (#17920)

There's no need to set execute permission bit for the checkpoint files.

Signed-off-by: Denis Barakhtanov <dbarahtanov@enakta.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants