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

DBZ-8564 Move away from dockerhub #105

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Conversation

vjuranek
Copy link
Member

@vjuranek vjuranek commented Jan 21, 2025

Integration tests use testcontainers Docker Compose which results into pulling docker/compose image. There doesn't seem to be a way how to force testconatiners to use different registry for this image, the only thing which seems to work is setting up env. variable TESTCONTAINERS_HUB_IMAGE_NAME_PREFIX, which would result into pulls from dockerhub when e.g. building it manually wihtout setting this variable, but this is probably not a big issue. At least CI should avoid using dockerhub, as TESTCONTAINERS_HUB_IMAGE_NAME_PREFIX is now set in GH actions.

Integration tests use testcontainers Docker Compose which results into
pulling `docker/compose` image. There doesn't seem to be a way how to
force testconatiners to use different registry for this image, the only
thing which seems to work is setting up env. variable
`TESTCONTAINERS_HUB_IMAGE_NAME_PREFIX`, which would result into pulls
from dockerhub when e.g. building it manually wihtout setting this
variable, but this is probably not a big issue. At least CI should avoid
using dockerhub, as `TESTCONTAINERS_HUB_IMAGE_NAME_PREFIX` is now set in
GH actions.
@jpechane jpechane merged commit caa244b into debezium:main Jan 21, 2025
3 checks passed
@jpechane
Copy link
Collaborator

@vjuranek Applied, thanks

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