Skip to content

Run TestImportExtremelyLargeImageWorks in parallel#38480

Merged
AkihiroSuda merged 1 commit intomoby:masterfrom
thaJeztah:run_slow_test_parallel
Jan 6, 2019
Merged

Run TestImportExtremelyLargeImageWorks in parallel#38480
AkihiroSuda merged 1 commit intomoby:masterfrom
thaJeztah:run_slow_test_parallel

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Jan 3, 2019

This is a slow test, taking over two minutes to complete, so spin up a new daemon for this test and run it in parallel.

follow-up to #38473 (see #38473 (comment)); I'll rebase once that PR is merged

Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐯 on commit 4836f5969317f7be52d9c425e00f7b4c988306a8 👼

This is a slow test, taking over two minutes to complete,
so spin up a new daemon for this test and run it in parallel

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the run_slow_test_parallel branch from 4836f59 to ae3ca7b Compare January 4, 2019 00:52
@codecov
Copy link

codecov bot commented Jan 4, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@2b7e084). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #38480   +/-   ##
=========================================
  Coverage          ?   36.71%           
=========================================
  Files             ?      608           
  Lines             ?    45078           
  Branches          ?        0           
=========================================
  Hits              ?    16551           
  Misses            ?    26247           
  Partials          ?     2280

@thaJeztah
Copy link
Member Author

rebased to get rid of the commits from #38473

Copy link
Member

@yongtang yongtang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AkihiroSuda AkihiroSuda merged commit f523876 into moby:master Jan 6, 2019
@thaJeztah thaJeztah deleted the run_slow_test_parallel branch January 6, 2019 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants