Skip to content

diff: hide types.Any from clients#6832

Merged
dmcgowan merged 1 commit intocontainerd:mainfrom
kzys:imgcrypt-typeurl
Apr 21, 2022
Merged

diff: hide types.Any from clients#6832
dmcgowan merged 1 commit intocontainerd:mainfrom
kzys:imgcrypt-typeurl

Conversation

@kzys
Copy link
Copy Markdown
Member

@kzys kzys commented Apr 20, 2022

(Please review and merge containerd/imgcrypt#75 first)

This commit hides types.Any from the diff package's interface. Clients
(incl. imgcrypt) shouldn't aware about gogo/protobuf.

@k8s-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kzys kzys changed the title Imgcrypt typeurl Use typeurl.Any on diff and imgcrypt Apr 20, 2022
@kzys kzys force-pushed the imgcrypt-typeurl branch 4 times, most recently from c9c8fad to f8ef7c8 Compare April 20, 2022 08:12
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 20, 2022

Build succeeded.

@kzys kzys force-pushed the imgcrypt-typeurl branch from f8ef7c8 to 4944ab3 Compare April 20, 2022 16:55
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 20, 2022

Build succeeded.

@kzys kzys force-pushed the imgcrypt-typeurl branch from 4944ab3 to b6f0aea Compare April 20, 2022 22:01
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 20, 2022

Build succeeded.

@kzys kzys force-pushed the imgcrypt-typeurl branch from b6f0aea to 8357d6b Compare April 20, 2022 23:34
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 20, 2022

Build succeeded.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 21, 2022

Build succeeded.

@kzys kzys force-pushed the imgcrypt-typeurl branch 2 times, most recently from 67a1b7b to f817e15 Compare April 21, 2022 02:01
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 21, 2022

Build succeeded.

@kzys kzys force-pushed the imgcrypt-typeurl branch from f817e15 to 7ab08dc Compare April 21, 2022 02:36
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 21, 2022

Build succeeded.

@kzys kzys changed the title Use typeurl.Any on diff and imgcrypt diff: hide types.Any from clients Apr 21, 2022
This commit hides types.Any from the diff package's interface. Clients
(incl. imgcrypt) shouldn't aware about gogo/protobuf.

Signed-off-by: Kazuyoshi Kato <[email protected]>
@kzys kzys force-pushed the imgcrypt-typeurl branch from 7ab08dc to dfa6e87 Compare April 21, 2022 13:43
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 21, 2022

Build succeeded.

@kzys kzys marked this pull request as ready for review April 21, 2022 14:58
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

Copy link
Copy Markdown
Member

@mikebrow mikebrow 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 7f8506a into containerd:main Apr 21, 2022
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