Skip to content

Makito fy/daos 18381 #17313

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
shimizukko wants to merge 2 commits into from
Closed

Makito fy/daos 18381 #17313

shimizukko wants to merge 2 commits into from

Conversation

@shimizukko
Copy link
Contributor

@shimizukko shimizukko commented Dec 25, 2025

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

Nasf-Fan and others added 2 commits December 24, 2025 13:17
@Nasf-Fan
DAOS-18381 ddb: properly transfer ddb cmdline options
1f47954 
The db_path maybe not contained inside the vos_path parameter,
especially under md-on-ssd mode. Related go interfaces need to
properly transfer cmdline options (db_path, write_mode, and so
on) from control plane to the lower ddb utils.

The patch also adds more check for the vos pool open and close
status before real VOS operation.

Signed-off-by: Fan Yong <[email protected]>
@shimizukko
Merge branch 'master' into makito-fy/DAOS-18381
131fbd9 
Skip-unit-tests: true
Skip-fault-injection-test: true
@github-actions
Copy link

github-actions bot commented Dec 25, 2025

Errors are component not formatted correctly,Ticket number prefix incorrect,PR title is malformatted. See https://daosio.atlassian.net/wiki/spaces/DC/pages/11133911069/Commit+Comments,Unable to load ticket data
https://daosio.atlassian.net/browse/Makito

@daosbuild3
Copy link
Collaborator

daosbuild3 commented Dec 25, 2025

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-17313/1/execution/node/926/log

@shimizukko
Copy link
Contributor Author

shimizukko commented Dec 29, 2025 •
edited
Loading

This PR is created to:

  • Include DAOS-18381 ddb: properly transfer ddb cmdline options #17305 so that rm_pool works.
  • PR17305 above doesn't contain the latest master that includes the fix of rdb-pool not being loaded with prov_mem. However, Fan Yong didn't want to merge master because some tests aren't passing in CI. Thus, I merged master here.

Note, ddb ls doesn't work with this PR.

@shimizukko shimizukko closed this Jan 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shimizukko @daosbuild3 @Nasf-Fan