Skip to content

[PROF-16425] update test container images - #933

Draft
kaahos wants to merge 5 commits into
mainfrom
paul.fournillon/fix-prof-16425-container-clang17
Draft

kaahos wants to merge 5 commits into
mainfrom
paul.fournillon/fix-prof-16425-container-clang17

Conversation

@kaahos

@kaahos kaahos commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?:
Use matched Clang 17, compiler-rt and llvm-cov defaults for local glibc images on x64 and ARM64. Version both cache levels with -clang17-cov1; preserve musl and dependent-JDK rebuild behavior.

Motivation:
Avoid Clang 14.0.0’s ARM64 sanitizer miscompilation of valid masked array indices. Keep coverage tools compatible with the default compiler.

Additional Notes:

  • Passed: agentic review, no findings.
  • Blocked: rebuilt local-image qualification, including installed toolchain identities, native/Java tests and coverage reporting. CI uses separate images.

How to test the change?:

bash utils/tests/test_run_containers_tests.sh
# Inside a provisioned Linux glibc image:
bash utils/tests/manual/test_masked_index_compiler.sh clang++
  • Passed locally: four focused configurations; supplemental 205 runner checks/27 cases and seven negative controls; formatting, syntax, source and documentation checks.
  • Failed in CI: test-matrix / test-linux-glibc-amd64 (11, debug, regular).

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a security review (run the dd:platform-security-review
    skill, or file a request via the PSEC review form).
    bewaire also runs automatically on every PR.
  • This PR doesn't touch any of that.
  • JIRA: PROF-16425

@kaahos kaahos changed the title [PROF-16425] Default local glibc test images to Clang 17 [PROF-16425] update test container images Nov 3, 2026
@dd-octo-sts

dd-octo-sts Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

✅ All 40 integration tests passed

📊 Dashboard · 👷 Pipeline · 📦 36576574

@dd-octo-sts

dd-octo-sts Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Reliability & Chaos Results

✅ All reliability & chaos checks passed Pipeline: https://gitlab.ddbuild.io/DataDog/java-profiler/-/pipelines/149885040

@dd-octo-sts

dd-octo-sts Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Scan-Build Report

User:runner@runnervmwuk1x
Working Directory:/home/runner/work/java-profiler/java-profiler/ddprof-lib/src/test/make
Command Line:make -j4 all
Clang Version:Ubuntu clang version 21.1.8 (6ubuntu1)
Date:Mon Nov 9 23:24:17 2026

Bug Summary

Bug TypeQuantityDisplay?
All Bugs2
Logic error
Dereference of null pointer1
Unused code
Dead initialization1

Filter Results by File

Reports

Bug Group Bug Type ▾ File Function/Method Line Path Length
Unused codeDead initializationreferenceChains.cppapplyOomRamp4971
Logic errorDereference of null pointerperfEvents_linux.cppunregisterThread72026

@dd-octo-sts

dd-octo-sts Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

CI Test Results

Run: #42719976827 | Commit: bf0ac4f | Duration: 19m 57s (longest job)

✅ All 32 test jobs passed

Status Overview

JDK glibc-aarch64/debug glibc-amd64/debug musl-aarch64/debug musl-amd64/debug
8 - ✅ - -
8-ibm - ✅ - -
8-j9 ✅ ✅ - -
8-librca - - ✅ ✅
8-orcl - ✅ - -
11 - ✅ - -
11-j9 ✅ ✅ - -
11-librca - - ✅ ✅
17 ✅ ✅ - -
17-graal ✅ ✅ - -
17-j9 ✅ ✅ - -
17-librca - - ✅ ✅
21 ✅ ✅ - -
21-graal ✅ ✅ - -
21-librca - - ✅ ✅
25 ✅ ✅ - -
25-graal ✅ ✅ - -
25-librca - - ✅ ✅

Legend: ✅ passed | ❌ failed | ⚪ skipped | 🚫 cancelled

Summary: Total: 32 | Passed: 32 | Failed: 0


Updated: 2026-11-09 23:45:05 UTC

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant