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

feat(#640): Add WithResourceMapping #652

Merged

Conversation

HofmeisterAn
Copy link
Collaborator

What does this PR do?

Adds WithResourceMapping to the Docker container builder pattern. The builder method allows to copy files or any binary contents to a created Docker container instance (that is not running).

Why is it important?

Some containers require certain files or configurations before they get started. WithResourceMapping allows to add these files or configurations up front. E.g. we copy the Elasticsearch JVM options to the container before it starts.

Related issues

Follow-ups

Add a more efficient way to bulk copy multiple resource mappings.

@HofmeisterAn HofmeisterAn added the enhancement New feature or request label Nov 4, 2022
@netlify
Copy link

netlify bot commented Nov 4, 2022

Deploy Preview for testcontainers-dotnet ready!

Name Link
🔨 Latest commit 21a3abc
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-dotnet/deploys/63652fb37a582400096d65c4
😎 Deploy Preview https://deploy-preview-652--testcontainers-dotnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@vlaskal
Copy link
Contributor

vlaskal commented Nov 4, 2022

@HofmeisterAn Nice increment.

@HofmeisterAn HofmeisterAn merged commit 99ac9a5 into develop Nov 5, 2022
@HofmeisterAn HofmeisterAn deleted the feature/631-support-copying-files-into-created-container branch November 5, 2022 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All Elasticsearch module tests fail
2 participants