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

chore: add caching to Linux docker run #123

Merged
merged 2 commits into from
Oct 9, 2024

Conversation

stevenh
Copy link
Contributor

@stevenh stevenh commented Mar 14, 2024

Add cache mount directives to Linux docker container so that reruns are much quicker.

@stevenh stevenh requested a review from a team as a code owner March 14, 2024 08:57
@mdelapenya
Copy link
Member

Hey @stevenh what are the motivation for this PR? Do you think it makes a real difference with the current state of the release process?

IIUC this will happen at build time of Ryuk, which is done just once per release. I see the benefits of building a Docker image faster, but want to check with you the implications/importance of this update.

Thanks!

@stevenh
Copy link
Contributor Author

stevenh commented Apr 8, 2024

For the release process no benefit. For the developer working on this it was much quicker iteration time, which was the motivation.

@stevenh stevenh force-pushed the chore/mount-caching branch from 9c2cf45 to e54e2a2 Compare October 4, 2024 11:18
Add cache mount directives to Linux docker container so that reruns
are much quicker.
@stevenh stevenh force-pushed the chore/mount-caching branch from e54e2a2 to c070a46 Compare October 4, 2024 11:19
Copy link
Member

@kiview kiview left a comment

Choose a reason for hiding this comment

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

I think this is in general a good and modern Dockerfile / docker build approach and considered best practice.

Copy link
Member

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@mdelapenya mdelapenya merged commit d0c3b50 into testcontainers:main Oct 9, 2024
9 checks passed
@mdelapenya mdelapenya self-assigned this Oct 9, 2024
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes that do not impact the existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants