Skip to content

DAOS-18688 client: Fix Lustre detection logic with the ASAN lib#17756

Open
knard38 wants to merge 1 commit intomasterfrom
ckochhof/fix/master/daos-18688/patch-001
Open

DAOS-18688 client: Fix Lustre detection logic with the ASAN lib#17756
knard38 wants to merge 1 commit intomasterfrom
ckochhof/fix/master/daos-18688/patch-001

Conversation

@knard38
Copy link
Contributor

@knard38 knard38 commented Mar 23, 2026

Description

This patch corrects a bug where Lustre was not detected if libasan was enabled via the SANITIZERS variable. The detection logic now works correctly regardless of sanitizer settings.

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

This patch corrects a bug where Lustre was not detected if libasan was
enabled via the SANITIZERS variable. The detection logic now works
correctly regardless of sanitizer settings.

Signed-off-by: Cedric Koch-Hofer <cedric.koch-hofer@hpe.com>
@knard38 knard38 self-assigned this Mar 23, 2026
@knard38 knard38 added the bug label Mar 23, 2026
@github-actions
Copy link

Ticket title is 'daos fails to compile when lustre is installed'
Status is 'In Progress'
Labels: 'ORNL'
https://daosio.atlassian.net/browse/DAOS-18688

@knard38 knard38 marked this pull request as ready for review March 23, 2026 19:51
@knard38 knard38 requested review from a team as code owners March 23, 2026 19:51
@knard38 knard38 requested review from daltonbohning and removed request for daltonbohning March 23, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants