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

Move integration tests to single execution containers in .drone.yml #1487

Merged
merged 2 commits into from
Sep 22, 2016

Conversation

LukasReschke
Copy link
Member

@LukasReschke LukasReschke commented Sep 22, 2016

Should make the execution faster in general.

@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @MorrisJobke and @rullzer to be potential reviewers

@LukasReschke LukasReschke force-pushed the move-integration-tests-to-single-containers branch from 97d95ca to d9c6779 Compare September 22, 2016 09:14
This adds improved performance

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschke LukasReschke force-pushed the move-integration-tests-to-single-containers branch from d9c6779 to cb19066 Compare September 22, 2016 09:26
@LukasReschke LukasReschke added the 3. to review Waiting for reviews label Sep 22, 2016
@LukasReschke LukasReschke changed the title [WIP] Move integration tests to single containers Move integration tests to single containers Sep 22, 2016
@LukasReschke LukasReschke changed the title Move integration tests to single containers Move integration tests to single execution containers in .drone.yml Sep 22, 2016
@LukasReschke LukasReschke added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Sep 22, 2016
@LukasReschke
Copy link
Member Author

I'll try to cache the dependencies for the integration tests in a docker image.

This one caches the dependencies

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschke LukasReschke added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 22, 2016
@LukasReschke
Copy link
Member Author

Added a container that has the dependencies cached.

@nickvergessen
Copy link
Member

still passes 👍

@MorrisJobke
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants