Skip to content

Public ImageData class for custom image pull policies#2585

Merged
bsideup merged 3 commits intotestcontainers:masterfrom
cha55son:bugfix/public-image-data
Apr 19, 2020
Merged

Public ImageData class for custom image pull policies#2585
bsideup merged 3 commits intotestcontainers:masterfrom
cha55son:bugfix/public-image-data

Conversation

@cha55son
Copy link
Copy Markdown
Contributor

I was following the docs here: https://www.testcontainers.org/features/advanced_options/#image-pull-policy for creating a custom pull policy. It wouldn't let me compile since the ImageData class was not public. This PR makes that class public and provides a test to ensure ImageData can be accessed outside the org.testcontainers.images package.

policies. Adding test to ensure it works as expected.
@rnorth
Copy link
Copy Markdown
Member

rnorth commented Apr 18, 2020

@cha55son we had some CI problems which are fixed on master, so I've updated the branch! Thanks.

@bsideup bsideup added this to the next milestone Apr 18, 2020
container image for the out of package pull policy test.
@bsideup bsideup merged commit b6cd9c8 into testcontainers:master Apr 19, 2020
quincy pushed a commit to quincy/testcontainers-java that referenced this pull request May 28, 2020
…iners#2585)

* Making imagedata public so it can be used to create custom image pull
policies. Adding test to ensure it works as expected.

* Ensuring we don't do unnecessary pulls and using the default generic
container image for the out of package pull policy test.

Co-authored-by: Chason Choate <[email protected]>
Co-authored-by: Richard North <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants