Skip to content

DAOS-18634 control: Fix container query for non-POSIX#17673

Open
tanabarr wants to merge 2 commits intomasterfrom
tanabarr/control-cont-print-fix
Open

DAOS-18634 control: Fix container query for non-POSIX#17673
tanabarr wants to merge 2 commits intomasterfrom
tanabarr/control-cont-print-fix

Conversation

@tanabarr
Copy link
Contributor

@tanabarr tanabarr commented Mar 9, 2026

In dmg container query workflow, remove recursive loop when trying to
print ObjectClass and avoid attempting to print POSIX container fields
if no embedded struct pointer exists. This fixes segfault when running
contain a query on a non-POSIX container. Add missing unit test
coverage for cmd/daos/pretty/container.go PrintContainers() and
PrintContainerInfo().

Features: control

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

Features: control
Signed-off-by: Tom Nabarro <thomas.nabarro@hpe.com>
@tanabarr tanabarr requested a review from liw March 9, 2026 23:49
@tanabarr tanabarr self-assigned this Mar 9, 2026
@github-actions
Copy link

github-actions bot commented Mar 9, 2026

Ticket title is 'daos container query p0 c1: panic: runtime error: invalid memory address or nil pointer dereference'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18634

@daosbuild3
Copy link
Collaborator

@daosbuild3
Copy link
Collaborator

Features: control
Signed-off-by: Tom Nabarro <thomas.nabarro@hpe.com>
@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-17673/1/execution/node/1364/log

@tanabarr tanabarr marked this pull request as ready for review March 11, 2026 11:36
@tanabarr tanabarr requested review from a team as code owners March 11, 2026 11:36
@tanabarr tanabarr requested review from kjacque and knard38 March 11, 2026 11:40
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