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

images: add pod infra container #3835

Closed
wants to merge 2 commits into from

Conversation

jelly
Copy link
Member

@jelly jelly commented Sep 6, 2022

Include the pod infra container in the test image required for testing
cockpit-podman's new create pod feature.

  • image-refresh debian-testing

Let's refresh debian-testing, as that's currently failing in this PR cockpit-project/cockpit-podman#961 with

https://cockpit-logs.us-east-1.linodeobjects.com/pull-961-20220902-183236-bfe5b802-debian-testing/log.html#13

> error: Pod failed to be created: Internal Server Error: could not pull image: initializing source docker://k8s.gcr.io/pause:3.5: pinging container registry k8s.gcr.io: Get "https://k8s.gcr.io/v2/": dial tcp: lookup k8s.gcr.io: no such host

Include the pod infra container in the test image required for testing
cockpit-podman's new create pod feature.
@jelly jelly requested a review from martinpitt September 6, 2022 19:59
@jelly jelly added the bot label Sep 6, 2022
@cockpituous
Copy link
Contributor

image-refresh in progress on 1-cockpit-7.
Log: https://logs.cockpit-project.org/logs/image-refresh-3835-20220906-200000/

@cockpituous cockpituous changed the title images: add pod infra container WIP: 1-cockpit-7: [no-test] images: add pod infra container Sep 6, 2022
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: 1-cockpit-7: [no-test] images: add pod infra container images: add pod infra container Sep 6, 2022
@martinpitt
Copy link
Member

LGTM! Please push the image and trigger tests.

@martinpitt martinpitt marked this pull request as draft September 9, 2022 13:19
@jelly
Copy link
Member Author

jelly commented Sep 13, 2022

Meanwhile debian-testing was refreshed in a new PR, so closing this one.

@jelly jelly closed this Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants