Skip to content

test: Increase btrfs loopback device size to 128MB#2448

Merged
mlaventure merged 1 commit intocontainerd:masterfrom
rn:btrfs
Jul 9, 2018
Merged

test: Increase btrfs loopback device size to 128MB#2448
mlaventure merged 1 commit intocontainerd:masterfrom
rn:btrfs

Conversation

@rn
Copy link
Copy Markdown
Contributor

@rn rn commented Jul 8, 2018

With btrfs-prog 4.17 as shipped with Alpine 3.8 the btrfs tests
fail with:

ERROR: '/dev/loop1' is too small to make a usable filesystem
ERROR: minimum size for each btrfs device is 114294784

btrfs-prog commit 997f9977c243 ("btrfs-progs: mkfs: Prevent
temporary system chunk to use space in reserved 1M range")
changed the code to compute the minimum size for a btrfs size
and is the likely reason for this error.

Increase the size of the loop back device to 128MB for
the btrfs test.

With this fix, the containerd tests pass on Alpine 3.8,
tested in LinuxKit with kernel 4.14.53.

fixes #2447

Signed-off-by: Rolf Neugebauer [email protected]

With btrfs-prog 4.17 as shipped with Alpine 3.8 the btrfs tests
fail with:

  ERROR: '/dev/loop1' is too small to make a usable filesystem
  ERROR: minimum size for each btrfs device is 114294784

btrfs-prog commit 997f9977c243 ("btrfs-progs: mkfs: Prevent
temporary system chunk to use space in reserved 1M range")
changed the code to compute the minimum size for a btrfs size
and is the likely reason for this error.

Increase the size of the loop back device to 128MB for
the btrfs test.

With this fix, the containerd tests pass on Alpine 3.8,
tested in LinuxKit with kernel 4.14.53.

fixes containerd#2447

Signed-off-by: Rolf Neugebauer <[email protected]>
@rn
Copy link
Copy Markdown
Contributor Author

rn commented Jul 8, 2018

If accepted, it would be good to cherry-pick this into 1.1. In LinuxKit we track the containerd releases and run the containerd tests on every PR (including kernel updates). I had to disable this LinuxKit test with the update to Alpine 3.8.

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2448   +/-   ##
=======================================
  Coverage   45.01%   45.01%           
=======================================
  Files          92       92           
  Lines        9414     9414           
=======================================
  Hits         4238     4238           
  Misses       4493     4493           
  Partials      683      683
Flag Coverage Δ
#linux 49.24% <ø> (ø) ⬆️
#windows 41.3% <ø> (ø) ⬆️

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 b416337...6519204. Read the comment docs.

@ehazlett
Copy link
Copy Markdown
Member

ehazlett commented Jul 9, 2018

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.

btrfs test fail btrfs-prog 4.17

6 participants