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.

Sign up for GitHub

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

Jump to bottom

speed up structure test #54

Merged

imjasonh merged 1 commit into chainguard-dev:main from imjasonh:stop-early

Jul 6, 2023

Member

imjasonh commented Jul 6, 2023

Skip mutate.Extract if there's only one layer (for our images, there will only be one layer). mutate.Extract is wasteful and blocks if there's only one layer. I've tested this works with a multi-layer image as well.
Stop reading the tar stream if all the file checks we plan to run have run.


          speed up structure test

9b54b57

Signed-off-by: Jason Hall <jason@chainguard.dev>

imjasonh requested a review from jonjohnsonjr

July 6, 2023 17:50

jonjohnsonjr approved these changes

View reviewed changes

imjasonh merged commit 7032e62 into chainguard-dev:main

8 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment