Skip to content

tests: Changed container image source for busybox#858

Merged
martymichal merged 1 commit into
containers:mainfrom
olivergs:toolbox-tests-busybox-quay
Aug 9, 2021
Merged

tests: Changed container image source for busybox#858
martymichal merged 1 commit into
containers:mainfrom
olivergs:toolbox-tests-busybox-quay

Conversation

@olivergs

@olivergs olivergs commented Aug 5, 2021

Copy link
Copy Markdown
Collaborator

Due to docker rate limiting we can not rely in docker.io for retrieving the images.

This was detected when executing our tests for podman fedora gating pipeline. Our busybox image was not downloaded and
one of the list tests was failing.

@softwarefactory-project-zuul

Copy link
Copy Markdown

Build failed.

@olivergs

olivergs commented Aug 5, 2021

Copy link
Copy Markdown
Collaborator Author

recheck

@softwarefactory-project-zuul

Copy link
Copy Markdown

Build succeeded.

@martymichal martymichal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The change looks alright. The reasoning behind switching from Docker to Quay is also fine. Apart from a single comment this looks good to go.

Comment thread images/test/busybox/Dockerfile
@martymichal martymichal added 2. Testing Related to testing of Toolbox - unit, system,.. 6. Minor Change Should not cause breakage labels Aug 9, 2021
Due to docker rate limiting we can not rely in docker.io for
retrieving the images.

This was detected when executing our tests for podman fedora
gating pipeline. Our busybox image was not downloaded and
one of the list tests was failing.
@olivergs olivergs force-pushed the toolbox-tests-busybox-quay branch from a06ae4b to 198d9b3 Compare August 9, 2021 14:48
@softwarefactory-project-zuul

Copy link
Copy Markdown

Build failed.

@martymichal martymichal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks alright. The test failure in Rawhide is unrelated.

@martymichal martymichal merged commit 09fb237 into containers:main Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. Testing Related to testing of Toolbox - unit, system,.. 6. Minor Change Should not cause breakage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants