Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: try pulling images using podman api directly #129

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

reubenmiller
Copy link
Collaborator

Due to a podman issue (experienced in podman 4.4), sometimes doing a docker pull fails for the following reason:

write /dev/stderr: input/output error

The PR adds logic to attempt to do a manual podman pull (via API) which does not seem to suffer the same problem.

This bug is most likely fixed in newer versions of podman (e.g. 5.x), though it is difficult for some users to update the podman versions.

Copy link

github-actions bot commented Feb 13, 2025

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
30 0 0 30 100 6m56.755003s

Passed Tests

Name ⏱️ Duration Suite
Get Container Logs 4.360 s Container-Logs
Get Container Logs with only last N lines 1.592 s Container-Logs
Remove Container 4.430 s Container-Remove
Remove Container Non Existent Container Should Not Through An Error 0.059 s Container-Remove
Restart Container 0.435 s Container-Restart
Restart Unknown Container Fails 0.066 s Container-Restart
Update to tedge-container-plugin-ng 25.039 s Installation
Check for Update 7.783 s Operations-Clone
Clone Existing Container 27.690 s Operations-Clone
Clone Existing Container by Timeout Whilst Waiting For Exit 18.060 s Operations-Clone
Clone Existing Container but Waiting For Exit 19.450 s Operations-Clone
Ignore Containers With Given Label 21.043 s Operations-Clone
Install/uninstall container package from private repository - credentials file 16.825 s Operations-Private-Registries
Install/uninstall container package from private repository - credentials script 16.759 s Operations-Private-Registries
Install/uninstall container package from private repository - credentials script with cache 21.727 s Operations-Private-Registries
Get Configuration 5.406 s Operations
Install/uninstall container-group package 15.773 s Operations
Install invalid container-group 3.071 s Operations
Install container-group with multiple files - app1 36.955 s Operations
Install container-group with multiple files - app2 10.219 s Operations
Install/uninstall container package 17.297 s Operations
Install/uninstall container package from file 12.063 s Operations
Manual container creation/deletion 20.898 s Operations
Manual container creation/deletion with error on run 8.061 s Operations
Manual container created and then killed 10.674 s Operations
Self Update Is Present Using Self Type 0.142 s Self
Self Update Is Present Using Container Type 0.147 s Self
Self Update Is Not Present 0.150 s Self
Service status 19.001 s Telemetry-Main
Sends measurements 71.582 s Telemetry-Main

@reubenmiller reubenmiller merged commit 6403819 into main Feb 13, 2025
4 checks passed
@reubenmiller reubenmiller deleted the feat-support-older-podman-pull-images branch February 13, 2025 13:24
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.

1 participant