Skip to content

chore(ci): fix ci images tags#682

Merged
kaahos merged 2 commits into
mainfrom
paul.fournillon/fix-ci-images
Jul 22, 2026
Merged

chore(ci): fix ci images tags#682
kaahos merged 2 commits into
mainfrom
paul.fournillon/fix-ci-images

Conversation

@kaahos

@kaahos kaahos commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?:

fix pinned image for slack-notifier and pin Alpine image to 3.22.1.

Motivation:

Some CI jobs fail due to security policies that require to pin every container image to an immutable version.

Additional Notes:

How to test the change?:

  • CI all green.

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: [JIRA-XXXX]

Unsure? Have a question? Request a review!

@dd-octo-sts

dd-octo-sts Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Benchmark Results (commit fbdb78d)

Pipeline: https://gitlab.ddbuild.io/DataDog/apm-reliability/benchmarking-platform/-/pipelines/126244881 Commit: fbdb78d2c19e5c5026b200cd2302697c10e2cc14

⚠️ Significant outliers

  • 🔴 fj-kmeans (JDK 21): runtime +6.6% (2672→2848 ms)
Runtime details (per benchmark × JDK)
Benchmark JDK Latest Dev Δ (dev vs latest) Issues L/D
akka-uct 21 ✅ 10314 ms (21 iters) ✅ 10384 ms (21 iters) ≈ +0.7% (±11.8%) — / —
akka-uct 25 ✅ 8900 ms (24 iters) ✅ 8892 ms (24 iters) ≈ -0.1% (±10.4%) — / —
finagle-chirper 21 ✅ 5983 ms (33 iters) ✅ 5967 ms (33 iters) ≈ -0.3% (±25.2%) ⚠️ W:3 / ⚠️ W:3
fj-kmeans 21 ✅ 2672 ms (69 iters) ✅ 2848 ms (66 iters) 🔴 +6.6% — / —
future-genetic 25 ✅ 2123 ms (88 iters) ✅ 2134 ms (87 iters) ≈ +0.5% (±2.4%) — / —
naive-bayes 21 ✅ 1281 ms (134 iters) ✅ 1245 ms (139 iters) ≈ -2.8% (±31.8%) — / —
naive-bayes 25 ✅ 1010 ms (169 iters) ✅ 1026 ms (167 iters) ≈ +1.6% (±31.7%) — / —
reactors 25 ✅ 18470 ms (15 iters) ✅ 18950 ms (15 iters) ≈ +2.6% (±4.5%) — / —
Internal counter details (ddprof)

ddprof internal counters, latest / dev (✅ = 0, · = unavailable):

Benchmark JDK Dropped rec Dropped jvmti Dropped trace Skipped WC AGCT fail Unwind fail
akka-uct 21 ✅ / ✅ ✅ / ✅ 2 / 1 2029 / 1995 ✅ / ✅ ✅ / ✅
akka-uct 25 ✅ / ✅ ✅ / ✅ 2 / 2 2080 / 2442 ✅ / ✅ ✅ / ✅
finagle-chirper 21 ✅ / ✅ ✅ / ✅ 2 / 1 8902 / 8451 ✅ / ✅ ✅ / ✅
finagle-chirper 25 ✅ / ✅ ✅ / ✅ 2 / 2 7986 / 8304 ✅ / ✅ ✅ / ✅
fj-kmeans 21 ✅ / ✅ ✅ / ✅ ✅ / 2 1243 / 1266 ✅ / ✅ ✅ / ✅
future-genetic 25 ✅ / ✅ ✅ / ✅ 1 / 1 2949 / 2920 ✅ / ✅ ✅ / ✅
naive-bayes 21 ✅ / ✅ ✅ / ✅ 6 / 1 3545 / 3529 ✅ / ✅ ✅ / ✅
naive-bayes 25 ✅ / ✅ ✅ / ✅ 2 / 2 3454 / 3502 ✅ / ✅ ✅ / ✅
reactors 21 ✅ / ✅ ✅ / ✅ ✅ / ✅ ✅ / ✅ ✅ / ✅ ✅ / ✅
reactors 25 ✅ / ✅ ✅ / ✅ 2 / ✅ 1882 / 1866 ✅ / ✅ ✅ / ✅

@kaahos
kaahos marked this pull request as ready for review July 22, 2026 14:24
@kaahos
kaahos requested a review from a team as a code owner July 22, 2026 14:24
@kaahos
kaahos merged commit ce0a824 into main Jul 22, 2026
103 of 105 checks passed
@kaahos
kaahos deleted the paul.fournillon/fix-ci-images branch July 22, 2026 14:34
@github-actions github-actions Bot added this to the 1.47.0 milestone Jul 22, 2026
@dd-octo-sts

dd-octo-sts Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

CI Test Results

Run: #29924609086 | Commit: c764926 | Duration: 3h 0m 16s (longest job)

1 of 32 test jobs failed

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

Failed Tests

glibc-aarch64/debug / 25-graal

Job: View logs

No detailed failure information available. Check the job logs.

Summary: Total: 32 | Passed: 30 | Failed: 1 | Cancelled: 1


Updated: 2026-07-22 16:39:32 UTC

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.

2 participants