Skip to content

Add preheat function for OCI image blob preheating via Dragonfly proxy#1774

Merged
gaius-qi merged 13 commits intomainfrom
copilot/add-preheat-function-again
Apr 10, 2026
Merged

Add preheat function for OCI image blob preheating via Dragonfly proxy#1774
gaius-qi merged 13 commits intomainfrom
copilot/add-preheat-function-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

  • Create new dragonfly-client-util/examples/request/ package with two binaries (get and preheat)
  • Create dragonfly-client-util/examples/request/Cargo.toml with [[bin]] entries for both get and preheat
  • Move get/src/main.rsrequest/src/bin/get.rs and preheat/src/main.rsrequest/src/bin/preheat.rs
  • Create a combined README.md for the request package
  • Remove old dragonfly-client-util/examples/get/ and dragonfly-client-util/examples/preheat/ directories
  • Update workspace Cargo.toml to replace the two old members with the new one
  • Verify lint and check passes

Copilot AI and others added 2 commits April 9, 2026 04:16
Add `oci-client` and `oci-spec` dependencies to workspace and util crate.
Create `request/preheat.rs` with `PreheatRequest` struct and `preheat`
function that:
- Parses OCI image reference and authenticates with registry
- Pulls manifest (with multi-platform index support)
- Downloads all blobs (config + layers) via Dragonfly proxy
- Discards content (preheat only caches in P2P network)

Agent-Logs-Url: https://github.com/dragonflyoss/client/sessions/f89011d2-1920-46d0-9bc5-48e1f8e086e8

Co-authored-by: gaius-qi <15955374+gaius-qi@users.noreply.github.com>
…d warn on missing reader

Agent-Logs-Url: https://github.com/dragonflyoss/client/sessions/f89011d2-1920-46d0-9bc5-48e1f8e086e8

Co-authored-by: gaius-qi <15955374+gaius-qi@users.noreply.github.com>
@gaius-qi gaius-qi added the enhancement New feature or request label Apr 9, 2026
@gaius-qi gaius-qi added this to the v2.5.0 milestone Apr 9, 2026
@gaius-qi gaius-qi marked this pull request as ready for review April 9, 2026 09:54
@gaius-qi gaius-qi requested review from a team as code owners April 9, 2026 09:54
@github-actions github-actions bot requested review from xujihui1985 and yyzai384 April 9, 2026 09:55
@gaius-qi gaius-qi enabled auto-merge (squash) April 9, 2026 09:55
…lure

When CI runs tests with --all-features, both ring and aws-lc-rs
features are enabled for rustls, which prevents auto-detection of
the CryptoProvider. Explicitly install ring as the default provider
at the start of the test.

Agent-Logs-Url: https://github.com/dragonflyoss/client/sessions/78ab4119-bb29-4144-99cd-b6a6b3bb7e4c

Co-authored-by: gaius-qi <15955374+gaius-qi@users.noreply.github.com>
auto-merge was automatically disabled April 9, 2026 10:21

Head branch was pushed to by a user without write access

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2026

Codecov Report

❌ Patch coverage is 0% with 157 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.41%. Comparing base (5400d49) to head (8b9c34f).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
dragonfly-client-util/src/request/mod.rs 0.00% 157 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1774      +/-   ##
==========================================
- Coverage   44.54%   44.41%   -0.14%     
==========================================
  Files          91       91              
  Lines       25834    26054     +220     
==========================================
+ Hits        11508    11571      +63     
- Misses      14326    14483     +157     
Files with missing lines Coverage Δ
dragonfly-client-util/src/request/mod.rs 32.82% <0.00%> (-11.07%) ⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…ng plugin example pattern

Agent-Logs-Url: https://github.com/dragonflyoss/client/sessions/efd37f6c-3bcc-4270-929b-ad5fd79cb709

Co-authored-by: gaius-qi <15955374+gaius-qi@users.noreply.github.com>
@gaius-qi gaius-qi enabled auto-merge (squash) April 10, 2026 07:00
Signed-off-by: Gaius <gaius.qi@gmail.com>
Copy link
Copy Markdown

@BraveY BraveY left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gaius-qi gaius-qi merged commit ce1f47f into main Apr 10, 2026
6 of 7 checks passed
@gaius-qi gaius-qi deleted the copilot/add-preheat-function-again branch April 10, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants