Skip to content

Conversation

@brandond
Copy link
Member

Proposed Changes

  • Bump spegel to upstream v0.5.1
  • Add oci.Store wrapper that allows deferring connection to the container runtime. Upstream spegel's oci.NewContainerd() now validates containerd features earlier, so we can no longer create the store and just wait to start the Tracker until containerd is up.

Types of Changes

version bump; removal of maintained fork

Verification

Check spegel version in go.mod

Testing

Yes

Linked Issues

User-Facing Change

Further Comments

@brandond brandond requested a review from a team as a code owner November 13, 2025 00:15
@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

❌ Patch coverage is 0% with 74 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.66%. Comparing base (e3cc61c) to head (e45ef15).

Files with missing lines Patch % Lines
pkg/spegel/store.go 0.00% 44 Missing ⚠️
pkg/spegel/spegel.go 0.00% 30 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13198      +/-   ##
==========================================
- Coverage   21.75%   21.66%   -0.10%     
==========================================
  Files         186      187       +1     
  Lines       15284    15349      +65     
==========================================
  Hits         3325     3325              
- Misses      11520    11585      +65     
  Partials      439      439              
Flag Coverage Δ
unittests 21.66% <0.00%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@brandond brandond force-pushed the bump-spegel branch 2 times, most recently from 77be282 to d4a3aae Compare November 13, 2025 03:30
@brandond
Copy link
Member Author

Should probably wait on spegel-org/spegel#1091 to be fixed first

dereknola
dereknola previously approved these changes Nov 17, 2025
Signed-off-by: Brad Davidson <[email protected]>
Spegel insists on checking containerd features when the store is created, so defer creating it until after contaienerd is up

Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
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