Skip to content

manage image inspect data in backend#44622

Merged
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:image_inspect_step1a
Dec 12, 2022
Merged

manage image inspect data in backend#44622
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:image_inspect_step1a

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

manage image inspect data in backend

This allows differentiating how the detailed data is collected between
the containerd-integration code and the existing implementation.

This PR is a subset of the changes in rumpl#27, and only implements the changes for the existing backend. Changes for the containerd-integration variant are done in #44621

thaJeztah and others added 2 commits December 9, 2022 19:00
Signed-off-by: Sebastiaan van Stijn <[email protected]>
This allows differentiating how the detailed data is collected between
the containerd-integration code and the existing implementation.

Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review area/images Image Service containerd-integration Issues and PRs related to containerd integration labels Dec 9, 2022
@thaJeztah thaJeztah added this to the v-next milestone Dec 9, 2022
@thaJeztah thaJeztah marked this pull request as ready for review December 9, 2022 21:42
@thaJeztah
Copy link
Copy Markdown
Member Author

@rumpl @corhere @tianon I was working on #43818 / #44621 and noticed that some squashing and reorganising may be needed, so I extracted the first (non-containers integration) changes of #44080) so that we can verify this doesn't affect the existing implementation, and so that the remaining changes are focussing on implementing things for the containerd-integration itself.

PTAL (after this is merged, I'll rebase the other one)

@thaJeztah
Copy link
Copy Markdown
Member Author

Thanks! Let me bring this one in 👍

@thaJeztah thaJeztah merged commit 487fa4a into moby:master Dec 12, 2022
@thaJeztah thaJeztah deleted the image_inspect_step1a branch December 12, 2022 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/images Image Service containerd-integration Issues and PRs related to containerd integration status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants