Skip to content

[release/1.4] re-enable unit tests in GitHub Actions environment#4763

Merged
fuweid merged 5 commits intocontainerd:release/1.4from
estesp:cp-tests-fix-1.4
Nov 24, 2020
Merged

[release/1.4] re-enable unit tests in GitHub Actions environment#4763
fuweid merged 5 commits intocontainerd:release/1.4from
estesp:cp-tests-fix-1.4

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Nov 23, 2020

Cherry-pick from #4719

Fixes runtime panic for testing snapshotters

Signed-off-by: Phil Estes <[email protected]>
(cherry picked from commit 027ee56)
GitHub Actions process wrapper sets score adj to 500 for any process;
the OOM score adj test expected default adj to be 0 during test.

Signed-off-by: Phil Estes <[email protected]>
(cherry picked from commit af2fb4e)
When running tests on any modern distro, this assumption will work. If
we need to make it work with kernels where we don't append this option
it will require some more involved changes.

Signed-off-by: Phil Estes <[email protected]>
(cherry picked from commit 85d9fe3)
Disable devmapper for now until test issues are fixed.

Signed-off-by: Phil Estes <[email protected]>
(cherry picked from commit c11472d)
Signed-off-by: Phil Estes <[email protected]>
(cherry picked from commit 6fb56aa)
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Nov 23, 2020

Build succeeded.

@containerd containerd deleted a comment from theopenlab-ci Bot Nov 23, 2020
Copy link
Copy Markdown
Contributor

@Zyqsempai Zyqsempai left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@fuweid fuweid merged commit 9c24574 into containerd:release/1.4 Nov 24, 2020
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