Skip to content

vendor: github.com/containerd/containerd v1.5.3 to reduce indirect dependencies#3154

Merged
silvin-lubecki merged 1 commit intodocker:masterfrom
thaJeztah:update_containerd_deps
Jul 15, 2021
Merged

vendor: github.com/containerd/containerd v1.5.3 to reduce indirect dependencies#3154
silvin-lubecki merged 1 commit intodocker:masterfrom
thaJeztah:update_containerd_deps

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Jun 22, 2021

follow-up to #3153

full diff: containerd/containerd@v1.5.2...v1.5.3

This removes the hcsshim dependency and other dependencies coming from that.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 22, 2021

Codecov Report

Merging #3154 (0ebdad7) into master (9262fad) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3154   +/-   ##
=======================================
  Coverage   58.57%   58.57%           
=======================================
  Files         299      299           
  Lines       21501    21501           
=======================================
  Hits        12594    12594           
  Misses       7984     7984           
  Partials      923      923           

…pendencies

full diff: containerd/containerd@v1.5.2...v1.5.3

This removes the hcsshim dependency and other dependencies coming from that.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the update_containerd_deps branch from 621b99c to 0ebdad7 Compare July 14, 2021 15:46
@thaJeztah thaJeztah changed the title [WIP] update containerd to remove Microsoft/hcsshim dependency vendor: github.com/containerd/containerd v1.5.3 to reduce indirect dependencies Jul 14, 2021
@thaJeztah thaJeztah marked this pull request as ready for review July 14, 2021 15:47
@thaJeztah
Copy link
Copy Markdown
Member Author

@silvin-lubecki @StefanScherer PTAL

@thaJeztah thaJeztah added this to the 21.xx milestone Jul 14, 2021
Copy link
Copy Markdown
Member

@StefanScherer StefanScherer 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
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

@silvin-lubecki silvin-lubecki merged commit f84487e into docker:master Jul 15, 2021
@thaJeztah thaJeztah deleted the update_containerd_deps branch July 15, 2021 12:31
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.

4 participants