Skip to content
This repository was archived by the owner on Mar 27, 2024. It is now read-only.

Run tests in parallel, dont use working dir #75

Merged
merged 1 commit into from
Sep 6, 2017

Conversation

r2d4
Copy link
Contributor

@r2d4 r2d4 commented Sep 6, 2017

This changes the CloudPrepper to not store intermediate artifacts in
the current directory, allowing the tests to be ran in parallel.

@aaron-prindle
Copy link
Collaborator

There is a '--save' flag for keeping the root file system which was previously in the current dir. Perhaps we should change that to save-dir and make this the default for the ImagePrepper?

This changes the CloudPrepper to not store intermediate artifacts in
the current directory, allowing the tests to be ran in parallel.
@r2d4
Copy link
Contributor Author

r2d4 commented Sep 6, 2017

I updated the PR to print the right directory with the --save flag. As for changing the name or the behavior, i think thats out of scope for this PR. As a first step, the message should probably be printed to stdout, instead of glog info'd.

@aaron-prindle
Copy link
Collaborator

LGTM

@aaron-prindle aaron-prindle merged commit 8761df3 into GoogleContainerTools:master Sep 6, 2017
@r2d4 r2d4 deleted the parallel branch September 6, 2017 20:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants