Skip to content

DAOS-19116 object: drop reference when fail in obj_ioc_begin_lite#18704

Open
Nasf-Fan wants to merge 1 commit into
masterfrom
Nasf-Fan/DAOS-19116_1
Open

DAOS-19116 object: drop reference when fail in obj_ioc_begin_lite#18704
Nasf-Fan wants to merge 1 commit into
masterfrom
Nasf-Fan/DAOS-19116_1

Conversation

@Nasf-Fan

Copy link
Copy Markdown
Contributor

It is possible that obj_ioc_begin_lite() maybe failed because of empty or stale pool map on server. At that time, the logic has already held reference against the container open handle. Under such case, directly returning error# to the caller may cause reference leak as to such container cannot be properly stopped in subsequent operations.

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

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

Ticket title is 'daos_racer/simple.py:DaosRacerSimpleTest.test_daos_racer_simple - Error running 'daos_racer'; timed out daos_racer process'
Status is 'In Progress'
Labels: '2.8.0rc1,2.8.0rc2,ci_2.8_provider,pr_test'
https://daosio.atlassian.net/browse/DAOS-19116

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@Nasf-Fan
Nasf-Fan force-pushed the Nasf-Fan/DAOS-19116_1 branch from eae3f7e to 32ecd41 Compare July 23, 2026 04:53
It is possible that obj_ioc_begin_lite() maybe failed because
of empty or stale pool map on server. At that time, the logic
has already held reference against the container open handle.
Under such case, directly returning error# to the caller may
cause reference leak as to such container cannot be properly
stopped in subsequent operations.

Signed-off-by: Fan Yong <fan.yong@hpe.com>
@Nasf-Fan
Nasf-Fan force-pushed the Nasf-Fan/DAOS-19116_1 branch from 32ecd41 to 8217ae2 Compare July 23, 2026 05:08
@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Unit Test completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-18704/2/display/redirect

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Unit Test bdev completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-18704/2/display/redirect

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Unit Test with memcheck completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-18704/2/display/redirect

@Nasf-Fan
Nasf-Fan marked this pull request as ready for review July 23, 2026 14:55
@Nasf-Fan
Nasf-Fan requested review from a team as code owners July 23, 2026 14:55
@Nasf-Fan
Nasf-Fan requested review from gnailzenh and liuxuezhao July 23, 2026 14:55
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