Skip to content

DAOS-18262 common: make sure DAOS file permissions are always 0XXY (#17680)#18175

Open
janekmi wants to merge 1 commit intorelease/2.8from
janekmi/DAOS-18262-0660-2.8
Open

DAOS-18262 common: make sure DAOS file permissions are always 0XXY (#17680)#18175
janekmi wants to merge 1 commit intorelease/2.8from
janekmi/DAOS-18262-0660-2.8

Conversation

@janekmi
Copy link
Copy Markdown
Contributor

@janekmi janekmi commented May 5, 2026 •
edited
Loading

Ref: #17680

At least:

  • 0660 for files
  • 0770 for directories.

So daos_server group can access DAOS files.

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

@janekmi
DAOS-18262 common: make sure DAOS file permissions are always 0XXY (#…
89af3e1 
…17680)

At least:

- 0660 for files
- 0770 for directories.

So daos_server group can access DAOS files.

Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>
@janekmi janekmi requested review from a team as code owners May 5, 2026 14:11
@janekmi janekmi added the clean-cherry-pick Cherry-pick from another branch that did not require additional edits label May 5, 2026
@janekmi janekmi requested review from Nasf-Fan and tanabarr May 5, 2026 14:13
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026 •
edited
Loading

Ticket title is 'dlck command seems to require sudo privileges. Also hit the DER_NOMEM (-1009) issue.'
Status is 'Awaiting backport'
Labels: 'request_for_2.8,scrubbed_2.8,test_2.8'
https://daosio.atlassian.net/browse/DAOS-18262

Nasf-Fan
Nasf-Fan reviewed May 6, 2026
View reviewed changes
Comment thread src/control/server/storage/bdev/backend_class.go
Comment thread src/mgmt/srv_target.c
@Nasf-Fan Nasf-Fan self-requested a review May 6, 2026 06:14
@janekmi janekmi requested a review from a team May 6, 2026 13:15
@daltonbohning daltonbohning changed the title DAOS-18262 common: make sure DAOS file permissions are always 0XXY DAOS-18262 common: make sure DAOS file permissions are always 0XXY (#17680) May 6, 2026
@daltonbohning daltonbohning added forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. waiting-for-merge-approval Waiting for merge approval labels May 6, 2026
@daltonbohning daltonbohning removed the request for review from a team May 6, 2026 14:51
@daltonbohning
Copy link
Copy Markdown
Contributor

daltonbohning commented May 6, 2026

Ticket does not yet have merge approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanabarr tanabarr tanabarr approved these changes

@kjacque kjacque kjacque approved these changes

@Nasf-Fan Nasf-Fan Nasf-Fan approved these changes

Assignees

No one assigned

Labels

clean-cherry-pick Cherry-pick from another branch that did not require additional edits forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. waiting-for-merge-approval Waiting for merge approval

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@janekmi @daltonbohning @tanabarr @kjacque @Nasf-Fan