Skip to content

[release/1.2 backport] Fix btrfs packages in contrib Dockerfile#3997

Merged
estesp merged 2 commits intocontainerd:release/1.2from
thaJeztah:1.2_backport_dockerfile_test_fixes
Jan 31, 2020
Merged

[release/1.2 backport] Fix btrfs packages in contrib Dockerfile#3997
estesp merged 2 commits intocontainerd:release/1.2from
thaJeztah:1.2_backport_dockerfile_test_fixes

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Building containerd inside a container showed that the Dockerfile was broken (due to the golang:1.12.x image now using Debian Buster as base image)

Nothing critical, but let's make the Dockerfile work again. Backports of:

rll2396 and others added 2 commits January 31, 2020 16:40
btrfs/ioctl.h is now included in libbtrfs-dev instead of btrfs-tools.
Update BUILDING.md Dockerfile to install the correct dependency.

Resolves: containerd#3813

Signed-off-by: Reid Li <[email protected]>
(cherry picked from commit a647407)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Phil Estes <[email protected]>
(cherry picked from commit fda652b)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3997 into release/1.2 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/1.2    #3997   +/-   ##
============================================
  Coverage        44.19%   44.19%           
============================================
  Files              100      100           
  Lines            10847    10847           
============================================
  Hits              4794     4794           
  Misses            5313     5313           
  Partials           740      740
Flag Coverage Δ
#linux 47.87% <ø> (ø) ⬆️
#windows 41% <ø> (ø) ⬆️

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 a558638...8761b1b. Read the comment docs.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit 9c7bd50 into containerd:release/1.2 Jan 31, 2020
@thaJeztah thaJeztah deleted the 1.2_backport_dockerfile_test_fixes branch February 1, 2020 06:50
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.

5 participants