Skip to content

go.mod: update containerd/imgcrypt, containerd/aufs, containerd/zfs#5185

Merged
dmcgowan merged 3 commits intocontainerd:masterfrom
thaJeztah:update_imgcrypt
Mar 13, 2021
Merged

go.mod: update containerd/imgcrypt, containerd/aufs, containerd/zfs#5185
dmcgowan merged 3 commits intocontainerd:masterfrom
thaJeztah:update_imgcrypt

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Mar 12, 2021

go.mod: github.com/containerd/imgcrypt 7ed62a527887

full diff: containerd/imgcrypt@0ae5c75...7ed62a5

go.mod: github.com/containerd/aufs v0.0.0-20210313051948-ffdde1490233

full diff: containerd/aufs@76a6863...ffdde14

go.mod: github.com/containerd/zfs v0.0.0-20210313052028-2233970b74b1

full diff: containerd/zfs@11e8f17...2233970

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Mar 12, 2021

Build succeeded.

@thaJeztah
Copy link
Copy Markdown
Member Author

let me include aufs and zfs bumps in this PR as well (go mod updates in those were now also merged)

@thaJeztah thaJeztah changed the title go.mod: github.com/containerd/imgcrypt 7ed62a527887 go.mod: update containerd/imgcrypt, containerd/aufs, containerd/zfs Mar 13, 2021
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Mar 13, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@dims dims left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Copy Markdown
Member Author

For imgcrypt, we should consider tagging a new release when we're closing GA (the other ones (aufs, zfs, btrfs) have not yet tagged releases)

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

@dmcgowan dmcgowan merged commit 1432839 into containerd:master Mar 13, 2021
@thaJeztah thaJeztah deleted the update_imgcrypt branch March 13, 2021 19:08
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