Revert merges with missing CI#1443
Conversation
…to v4 (open-telemetry#1410)" This reverts commit 0ff4c99.
This reverts commit 72e7f10.
This reverts commit fa476e5.
There was a problem hiding this comment.
Pull request overview
This PR reverts multiple base image upgrades that were merged without passing CI. The reversal brings Go, Node.js, Ruby, Java (Temurin), .NET, Alpine, Debian, MySQL, Redis, and Kafka images back to previously validated versions, aligning them with the go 1.25.7 directive in go.mod.
Changes:
- Downgrades all Go builder images from
golang:1.26.0back togolang:1.25.7across the entire codebase. - Reverts non-Go base images (Node.js, Ruby, Java/Temurin, .NET, Alpine, Debian, MySQL, Redis, Kafka) from versions that did not have CI coverage to previously validated versions.
- Replaces pinned
sha256digests to match the reverted image versions throughout all Dockerfiles.
Reviewed changes
Copilot reviewed 65 out of 65 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
k8scache.Dockerfile |
Reverts Go builder from 1.26.0 to 1.25.7 |
generator.Dockerfile |
Reverts Go alpine builder from 1.26.0 to 1.25.7 |
debug.Dockerfile |
Reverts Go alpine builder from 1.26.0 to 1.25.7 |
examples/otel-collector/Dockerfile |
Reverts Go builder and final Alpine image (3.23 → 3.19) |
internal/test/vm/Dockerfile |
Reverts Alpine base from 3.23 to 3.20 |
internal/test/integration/components/nodejsdist/Dockerfile_b |
Reverts Node.js from 24 to 18 (inconsistent with other Node Dockerfiles) |
internal/test/integration/components/nodejsdist/Dockerfile_r |
Reverts Node.js from 24 to 18 (inconsistent with other Node Dockerfiles) |
internal/test/integration/components/nodejsserver/Dockerfile |
Reverts Node.js from 24 to 20 |
internal/test/integration/components/nodeclient/Dockerfile |
Reverts Node.js from 24 to 20 |
internal/test/integration/components/nodemultiproc/Dockerfile |
Reverts Node.js from 24 to 20 |
internal/test/integration/components/tpclient/Dockerfile |
Reverts Node.js from 24 to 20 |
internal/test/integration/components/rubytestserver/testapi/Dockerfile |
Reverts Ruby from 4.0.1 to 3.4.8 |
internal/test/integration/components/rubytestserver/testapi/Dockerfile_tls |
Reverts Ruby from 4.0.1 to 3.4.8 |
internal/test/integration/components/nginx-rails-sql/testapi/Dockerfile |
Reverts Ruby from 4.0.1 to 3.4.8 |
internal/test/integration/components/javatestserver/Dockerfile_jar |
Reverts Temurin from 25 to 17 |
internal/test/integration/components/javakafka-tls/Dockerfile |
Reverts Temurin JRE from 25 to 21 |
internal/test/integration/components/java_tls/*/Dockerfile |
Reverts Temurin JRE from 25 to 21 |
internal/test/integration/components/dotnetserver/Dockerfile |
Reverts .NET from 10.0 to 7.0 |
internal/test/integration/components/testserver/Dockerfile_static |
Reverts Debian from 13 to 12 |
internal/test/integration/components/rustssltestserver/Dockerfile |
Reverts Debian from trixie to bookworm |
internal/test/integration/components/mysqldb/Dockerfile |
Reverts MySQL from 9 to 8 |
internal/test/integration/components/redis/Dockerfile |
Reverts Redis from 8.6.1 to 7.4.8 |
| Multiple kafka docker-compose files | Reverts bitnamilegacy/kafka from 4.0 to 3.9 |
| Multiple Go component Dockerfiles | Reverts Go builder from 1.26.0 to 1.25.7 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
skl
left a comment
There was a problem hiding this comment.
You would need to merge this to get the VM workflows to pass again
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1443 +/- ##
==========================================
- Coverage 44.89% 44.89% -0.01%
==========================================
Files 312 312
Lines 28901 28901
==========================================
- Hits 12976 12974 -2
- Misses 15133 15136 +3
+ Partials 792 791 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Uh oh!
There was an error while loading. Please reload this page.