Skip to content

ci: use pre-built CI container in coverage and test workflows#217

Merged
rst0git merged 1 commit into
checkpoint-restore:mainfrom
adrianreber:2026-04-20-ci-container
Apr 21, 2026
Merged

ci: use pre-built CI container in coverage and test workflows#217
rst0git merged 1 commit into
checkpoint-restore:mainfrom
adrianreber:2026-04-20-ci-container

Conversation

@adrianreber

Copy link
Copy Markdown
Member

Replace fedora:latest with the weekly built CI container image from ghcr.io and remove the dnf install steps, as all packages are already included in the container.

@github-actions

github-actions Bot commented Apr 20, 2026

Copy link
Copy Markdown

Test Results

90 tests  ±0   90 ✅ ±0   4s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit a86f696. ± Comparison against base commit eea83bf.

♻️ This comment has been updated with latest results.

@adrianreber

Copy link
Copy Markdown
Member Author

Package are not public:

 Public

Make this package visible to anyone.
Setting is disabled by organization administrators. 

and I cannot change it.

@rst0git

rst0git commented Apr 21, 2026

Copy link
Copy Markdown
Member

@adrianreber We can also use quay.io/criu/checkpointctl-ci. I've added username and secret to the checkpointctl repository.

@adrianreber

Copy link
Copy Markdown
Member Author

Thanks. It would be more efficient to use the ghcr registry as it is much closer and faster. We need an organization admin to allow public images. I guess Andrei should be able to do it.

@adrianreber

Copy link
Copy Markdown
Member Author

Wait, better idea coming.

@adrianreber adrianreber force-pushed the 2026-04-20-ci-container branch from 4c1c515 to 0af69f8 Compare April 21, 2026 07:38
@rst0git

rst0git commented Apr 21, 2026

Copy link
Copy Markdown
Member

It would be more efficient to use the ghcr registry as it is much closer and faster.

btw, I'm not sure if it is actually faster: https://github.com/orgs/community/discussions/27080

@codecov-commenter

codecov-commenter commented Apr 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.27%. Comparing base (6e607c0) to head (a86f696).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
+ Coverage   79.98%   84.27%   +4.28%     
==========================================
  Files          16       16              
  Lines        1759     1920     +161     
==========================================
+ Hits         1407     1618     +211     
+ Misses        263      224      -39     
+ Partials       89       78      -11     

☔ 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.

Replace fedora:latest with the weekly built CI container image
from ghcr.io and remove the dnf install steps, as all packages
are already included in the container.

Generated with Claude Code (https://claude.ai/code)

Signed-off-by: Adrian Reber <areber@redhat.com>
@adrianreber adrianreber force-pushed the 2026-04-20-ci-container branch from 0af69f8 to a86f696 Compare April 21, 2026 08:13
@rst0git rst0git merged commit 8dad90b into checkpoint-restore:main Apr 21, 2026
11 checks passed
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.

3 participants