ci: improved CI image hashing, multi-arch manifest, S3 snapshot uploads#36
Merged
pawelchcki merged 1 commit intomainfrom Feb 26, 2026
Merged
ci: improved CI image hashing, multi-arch manifest, S3 snapshot uploads#36pawelchcki merged 1 commit intomainfrom
pawelchcki merged 1 commit intomainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #36 +/- ##
=======================================
Coverage 63.12% 63.12%
=======================================
Files 7 7
Lines 377 377
Branches 49 49
=======================================
Hits 238 238
Misses 101 101
Partials 38 38 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
xlamorlette-datadog
approved these changes
Feb 4, 2026
Contributor
xlamorlette-datadog
left a comment
xlamorlette-datadog
left a comment
There was a problem hiding this comment.
LGTM.
The list-s3-artifacts job needs to be fixed:
$ aws s3 ls s3://rum-auto-instrumentation/httpd/ --recursive
Unable to locate credentials. You can configure credentials by running "aws configure".
Contributor
Author
|
/remove |
|
View all feedbacks in Devflow UI.
|
gh-worker-dd-devflow-36fce6
bot
added
mergequeue-status: removed
and removed
mergequeue-status: waiting
labels
pawelchcki
changed the title
pawelchcki
force-pushed
the
add_aws_automations
branch
9 times, most recently
from
c4ddf47 to
1a9451a
Compare
- Improved CI image hashing (includes Dockerfile, build_env, setup script) - Multi-arch manifest creation for CI images - Nydus conversion with proper already-nydusified detection - S3 snapshot uploads for branch and main pipelines - Integration test stage (test-rum) - Use upstream docker and aws-cli images directly
pawelchcki
force-pushed
the
add_aws_automations
branch
from
df1ade2 to
fa2c2d7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docker buildx imagetools createtest-rum) for RUM validationmod_datadog.so) to S3 with--acl public-read:httpd/branch-snapshots/pipeline-<id>/for feature brancheshttpd/snapshots/pipeline-<id>/+httpd/snapshots/latest/for maindocker:27.3.1andaws-cli:2.33.12images directly fromregistry.ddbuild.io/images/allow_failurefrombuild-ci-imageDOCKERFILE_HASH→CI_IMAGE_HASHTest plan
add_aws_automationsbranchhttpd/snapshots/andhttpd/snapshots/latest/