Skip to content

DAOS-18984 dfs: dfs_obj_set_oclass should update the dir open handle#18286

Merged
mchaarawi merged 1 commit into
masterfrom
mschaara/18984
May 21, 2026
Merged

DAOS-18984 dfs: dfs_obj_set_oclass should update the dir open handle#18286
mchaarawi merged 1 commit into
masterfrom
mschaara/18984

Conversation

@mchaarawi
Copy link
Copy Markdown
Contributor

@mchaarawi mchaarawi commented May 19, 2026

  • dfs_obj_set_oclass() changes the on-disk object class for children objects created in the directory, however misses updating that on the open handle of the directory.
  • Fix some tests to account for small systems.
  • EC oclass will not be chosen if there are not enough Fault Domains, so the dir oclass will actually use what we set on the parent dir instead of the container default.

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

Signed-off-by: Mohamad Chaarawi <mohamad.chaarawi@hpe.com>
@github-actions
Copy link
Copy Markdown

Ticket title is 'dfs_obj_set_oclass does not modify the oclass on the open handle'
Status is 'Open'
Labels: 'triaged'
https://daosio.atlassian.net/browse/DAOS-18984

@mchaarawi mchaarawi marked this pull request as ready for review May 19, 2026 21:18
@mchaarawi mchaarawi requested review from a team as code owners May 19, 2026 21:18
@mchaarawi mchaarawi requested a review from wangshilong May 20, 2026 12:56
@mchaarawi mchaarawi requested a review from a team May 21, 2026 02:54
@mchaarawi mchaarawi merged commit 7a7994d into master May 21, 2026
44 of 45 checks passed
@mchaarawi mchaarawi deleted the mschaara/18984 branch May 21, 2026 02:54
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.

3 participants