Skip to content

Conversation

@shimizukko
Copy link
Contributor

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 25, 2025 10:59
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.

Features: control recovery

Signed-off-by: Fan Yong <fan.yong@hpe.com>
Skip-unit-tests: true
Skip-fault-injection-test: true
@github-actions
Copy link

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

@shimizukko
Copy link
Contributor Author

shimizukko commented Dec 29, 2025

This PR is the combination of #17312 and master:

  • PR17312 has some updates related to ddb.
  • PR17312 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.

@daosbuild3
Copy link
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-17323/1/execution/node/940/log

Skip-unit-tests: true
Skip-fault-injection-test: true
Skip-test: true
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