Skip to content

[CI] remove extra testing on older Ubuntu#3642

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
dmcgowan:remove-duplicated-travis-tests
Sep 13, 2019
Merged

[CI] remove extra testing on older Ubuntu#3642
dmcgowan merged 1 commit intocontainerd:masterfrom
dmcgowan:remove-duplicated-travis-tests

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

Currently only 5 parallel Travis tests are done, the older LTS testing the full matrix is not necessary and worth the cost of the increased build time. Just run the test the latest supported runtime.

Testing the older LTS does not require testing the full matrix
to test for compatibility.

Signed-off-by: Derek McGowan <[email protected]>
@dmcgowan dmcgowan added this to the 1.3 milestone Sep 11, 2019
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 11, 2019

Build succeeded.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3642 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3642   +/-   ##
=======================================
  Coverage   42.39%   42.39%           
=======================================
  Files         127      127           
  Lines       14048    14048           
=======================================
  Hits         5955     5955           
  Misses       7193     7193           
  Partials      900      900
Flag Coverage Δ
#linux 45.9% <ø> (ø) ⬆️
#windows 37.27% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d3ccbc6...36c543a. Read the comment docs.

1 similar comment
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3642 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3642   +/-   ##
=======================================
  Coverage   42.39%   42.39%           
=======================================
  Files         127      127           
  Lines       14048    14048           
=======================================
  Hits         5955     5955           
  Misses       7193     7193           
  Partials      900      900
Flag Coverage Δ
#linux 45.9% <ø> (ø) ⬆️
#windows 37.27% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d3ccbc6...36c543a. Read the comment docs.

@thaJeztah
Copy link
Copy Markdown
Member

curious; it's only running on master, so where is it causing delays?

@dmcgowan
Copy link
Copy Markdown
Member Author

dmcgowan commented Sep 11, 2019

@thaJeztah verifying master and release builds

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit 8ab81a0 into containerd:master Sep 13, 2019
@dmcgowan dmcgowan deleted the remove-duplicated-travis-tests branch March 23, 2022 22:26
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.

4 participants