Skip to content

use registry-image as the docker-image resourse is legacy#641

Merged
jochenehret merged 2 commits intocloudfoundry:mainfrom
ramonskie:main
Mar 31, 2026
Merged

use registry-image as the docker-image resourse is legacy#641
jochenehret merged 2 commits intocloudfoundry:mainfrom
ramonskie:main

Conversation

@ramonskie
Copy link
Copy Markdown
Contributor

the registry-image is replacement for docker-image resource
and the docker-image resources now causes allot of captcha errors to dockerhub

Copy link
Copy Markdown
Contributor

@jochenehret jochenehret left a comment

Choose a reason for hiding this comment

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

Basically ok, but in a few tasks we likely need to keep the "docker-image" resource for building images.


- name: relint-base-image
type: docker-image
type: registry-image
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the "registry-image" resource canNOT be used for building Docker images, only for pulling and pushing. So for the "build-docker-images" pipeline, we should stick to "docker-image"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

you are correct.

This resource does not and will never support building - only registry image pushing/pulling. Building should instead be done with something like the oci-build task (or anything that can produce OCI image tarballs).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

reverted the changes for the build pipeline

@jochenehret jochenehret requested review from a team and jochenehret March 31, 2026 12:20
@jochenehret jochenehret merged commit 3f78243 into cloudfoundry:main Mar 31, 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.

2 participants