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.

Sign up for GitHub

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

Jump to bottom

Docker compose support #25

Closed
mraerino opened this issue Nov 22, 2018 · 3 comments · Fixed by #97
Closed

Docker compose support #25

mraerino opened this issue Nov 22, 2018 · 3 comments · Fixed by #97

Comments

@mraerino
Copy link
Contributor

The java implementation supports spinning up docker compose setups to test larger setups as a black box.

This is interesting for this library especially since many DevOps people are using Golang for their services anyways.
@mdelapenya
Copy link
Collaborator

Is there a plan to work on this? I'd be willing to help in this efforts if possible

@gianarb
Copy link
Collaborator

gianarb commented Sep 10, 2019 via email

@mdelapenya
Copy link
Collaborator

mdelapenya commented Sep 10, 2019 •
edited

Yeah, I'm currently playing around with the local docker compose implementation (no socat)

mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 11, 2019
@mdelapenya
(#25) Add utility to generate random strings
1788659
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 11, 2019
@mdelapenya
(#25) Create very basic implementation of Local Docker Compose implem…
d905a32 
…entation
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 11, 2019
@mdelapenya
(#25) Add a function to "down" the compose
1c369fe
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 11, 2019
@mdelapenya
(#25) Add a function to "down" the compose
0f66b80
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 11, 2019
@mdelapenya
(#25) Make ExecError public
b8622f4
@mdelapenya mdelapenya mentioned this issue Sep 11, 2019
Merged
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 11, 2019
@mdelapenya
(#25) Make ExecError public
38810f1
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Use lowercase for container/service identifiers
94069c6
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Fix compose down
501c6a0 
We need to add compose-specific environment variables for identifier
and compose file path
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Add a test verifying that the container receives the env vars
c563b90
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Extract assertion of container env to a function
f9d9364 
We reuse it in the first test, checking that the env variable is not set
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Extract default environment generation to a function
6b395a9
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Extract compose execution to a function
f7027b2 
It is shared by invoke and down
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Support passing multiple docker-compose files
a6d823e
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Support checking for absent env vars in tests
1d0b640
mdelapenya referenced this issue in mdelapenya/testcontainers-go Sep 18, 2019
@mdelapenya
(#25) Make tests more readable
12d23c3
mdelapenya referenced this issue in mdelapenya/testcontainers-go Oct 29, 2019
@mdelapenya
(#25) Remove unneeded log
dfb14b0 
It could disclose environment variables
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 18, 2019
@mdelapenya
(#25) Use Google's UUID for generating compose's identifier
72e1dd5
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 18, 2019
@mdelapenya
(#25) Add local compose to the docs
054d961 
Referenced from https://www.testcontainers.org/modules/docker_compose/
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 18, 2019
@mdelapenya
(#25) Rename simple compose test resource
82a0ecd
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 18, 2019
@mdelapenya
(#25) Add a test with a more complex scenario
a62b618
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 18, 2019
@mdelapenya
(#25) Fix path in test
5dea265
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 19, 2019
@mdelapenya
(#25) Add info about the command and the arguments when compose fails
ca46331 
I'm not adding information about the environment to avoid disclosing
sensible values, but it could be easily added
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 19, 2019
@mdelapenya
(#25) Add logs to the tests
ba29b24
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 19, 2019
@mdelapenya
WIP (#25) Add logs to the tests
b426008
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 20, 2019
@mdelapenya
(#25) Create defer functions the soonest
cda34df
mdelapenya referenced this issue in mdelapenya/testcontainers-go Nov 22, 2019
@mdelapenya
(#25) Add utility to generate random strings
54d8825
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Fix compose down
06ada3c 
We need to add compose-specific environment variables for identifier
and compose file path
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Add a test verifying that the container receives the env vars
a320e66
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Extract assertion of container env to a function
1badb29 
We reuse it in the first test, checking that the env variable is not set
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Extract default environment generation to a function
47362c1
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Extract compose execution to a function
d9e1579 
It is shared by invoke and down
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Support passing multiple docker-compose files
2604294
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Support checking for absent env vars in tests
4807a91
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Make tests more readable
80f3758
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Remove unneeded log
7614627 
It could disclose environment variables
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Use Google's UUID for generating compose's identifier
28b4ce4
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Add local compose to the docs
866cd2a 
Referenced from https://www.testcontainers.org/modules/docker_compose/
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Rename simple compose test resource
26eee98
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Add a test with a more complex scenario
8096d05
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Fix path in test
efa9db2
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Add info about the command and the arguments when compose fails
3c26102 
I'm not adding information about the environment to avoid disclosing
sensible values, but it could be easily added
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Create defer functions the soonest
a07447a
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Add comments and Go Examples for docs
a3e1f0b
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Do not override ports on Travis CI
ef1c203
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Support getting the services from the compose files
71af051
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Execute go mod tidy
956a03b
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Provide more information when failing a Compose test
446944d
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Extract array population on exec errors
b587960
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Enhance exec error
661033b
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Optimise reading for docker-compose output
88efbd0 
Read from: https://blog.kowalczyk.info/article/wOYk/advanced-command-execution-in-go-with-osexec.html
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Do not assert the environment variables in the simple test
9570dd9
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Fix tests reading environment variables from the compose file
7182ddc 
It was just a matter of setting the container name at docker-compose
level, because we just want to access the container to read its
environment variables and do some calculations with them.
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Add troubleshooting Travis guide
6830723
mdelapenya referenced this issue in mdelapenya/testcontainers-go Apr 16, 2020
@mdelapenya
(#25) Run go mod tidy
03c0820
gianarb added a commit that referenced this issue Apr 20, 2020
@gianarb
Merge pull request #97 from mdelapenya/25-docker-compose
a911c8e 
(#25) Create very basic implementation of Local Docker Compose
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

(#25) Create very basic implementation of Local Docker Compose mdelapenya/testcontainers-go
3 participants
@mdelapenya @gianarb @mraerino