Skip to content
This repository has been archived by the owner on May 18, 2020. It is now read-only.

Make integration test reusable #22

Open
yurishkuro opened this issue Jul 27, 2017 · 1 comment
Open

Make integration test reusable #22

yurishkuro opened this issue Jul 27, 2017 · 1 comment

Comments

@yurishkuro
Copy link
Member

I was looking at BaseETest.java in light of the docker-compose question https://github.com/uber/jaeger/issues/286.

It seems the test is a fairly generic end-to-end check of "is my Jaeger installation wired and working correctly" (@black-adder - didn't we have a similar one in the main repo?)

My question is - why not implement that test in the main jaeger repo, in Go, and build it as a Docker image, with some parameters controlled via env variables? Then if issue 286 results in a docker-compose file in the repo, it can be tested by adding the e2e as the last step, and k8s and openshift templates can also be tested with the same docker image.

@pavolloffay
Copy link
Member

What we can also do is to reuse that new compose file for xdock tests. Xdock compose file would just simply extend the default docke-compose deployment (e.g. docker-compose up -f default-compose.yml -f xdock-compose.yml) and add crossdock, test_driverand xdock-langs. This way we would not have to test that new compose file separately. We did very similar thing in hawkular-apm instrumentation-framework.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants