Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

image inspect fixes & cleanup #3008

Closed
wants to merge 1 commit into from

Conversation

apostasie
Copy link
Contributor

Overall, this enhances image inspect:

  • multiple images with the same digest are now coalesced properly into one, with multiple RepoTags
  • multiple images resulting for multiple arguments in the call to image inspect will coalesce into one array, instead of a stream of arrays
  • ImageFromNative has been cleaned-up and simplified, making room for later addition of currently missing properties

In the process, we stopped calling on the image Walker which serves little to no purpose here, hence significantly reducing the complexity.

This mainly fixes #3004 and part of #3006.

After this, more work is needed overall for inspect across objects, and then some specifically for image inspect.

@apostasie
Copy link
Contributor Author

hey @AkihiroSuda

Most of my PRs are failing the CIs.
Looks like:

#85 1.141 cni-plugins-linux-amd64-v1.4.1.tgz: FAILED
#85 1.142 sha256sum: WARNING: 1 of 1 computed checksums did NOT match

If this is it ^, did the cni-plugins folks do something messy?

@AkihiroSuda
Copy link
Member

Asking in:

Signed-off-by: apostasie <spam_blackhole@farcloser.world>
@apostasie
Copy link
Contributor Author

@AkihiroSuda I am closing this.

There is a lot more work needed to fix image inspect which I am currently working on.
I'll reopen a more thorough PR (hopefully with tests) once I am fully done.

@apostasie apostasie closed this May 16, 2024
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.

Current implementation of images inspect is faulty (eg: RepoTags)
2 participants