Skip to content

Run CI with temporary files on a ramdisk - #2254

Merged
cyli merged 1 commit into
moby:masterfrom
aaronlehmann:tmpfs
Jun 14, 2017
Merged

Run CI with temporary files on a ramdisk#2254
cyli merged 1 commit into
moby:masterfrom
aaronlehmann:tmpfs

Conversation

@aaronlehmann

Copy link
Copy Markdown
Contributor

This sets up a tmpfs mount to be used for temporary files created by tests.

If this works, hopefully it will fix the excessive I/O latencies that
are blocking tests for long periods and causing them to fail.

As a side effect, this also stores temporary compilation artifacts on
tmpfs.

@codecov

codecov Bot commented Jun 14, 2017

Copy link
Copy Markdown

Codecov Report

Merging #2254 into master will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2254      +/-   ##
=========================================
+ Coverage   60.31%   60.7%   +0.39%     
=========================================
  Files         124     124              
  Lines       20263   20263              
=========================================
+ Hits        12221   12301      +80     
+ Misses       6654    6578      -76     
+ Partials     1388    1384       -4

This sets up a tmpfs mount to be used for temporary files created by tests.

If this works, hopefully it will fix the excessive I/O latencies that
are blocking tests for long periods and causing them to fail.

As a side effect, this also stores temporary compilation artifacts on
tmpfs.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
@aaronlehmann

Copy link
Copy Markdown
Contributor Author

Well, it seems to work. I ran CI 5 times or so. Should we merge this and see what happens?

@aaronlehmann aaronlehmann changed the title [WIP] Run CI with temporary files on a ramdisk Run CI with temporary files on a ramdisk Jun 14, 2017
@cyli

cyli commented Jun 14, 2017

Copy link
Copy Markdown
Contributor

LGTM! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants