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

fix: filter platforms when set metric ImageSize #134

Merged
merged 2 commits into from
May 12, 2023

Conversation

Desiki-high
Copy link
Collaborator

When we pull the image, we filter the platform, but the index descriptor will include all platforms. In that case, when we need to calculate the SourceImageSize, we will get error platform manifests. We can't find blobs locally.

When we pull the image, we filte the platform, but the index descriptor will include
all platforms. In that case, when we need to calculate the SourceImageSize, we will get
error platform manifests. We can't find blobs locally because we do filter in pull.

Signed-off-by: Yadong Ding <ding_yadong@foxmail.com>
@Desiki-high Desiki-high force-pushed the fix-source-platform branch from 273ba91 to b9d2710 Compare May 12, 2023 03:07
@Desiki-high Desiki-high changed the title fix: filter platforms when set metric SourceImageSize fix: filter platforms when metric ImageSize May 12, 2023
@Desiki-high Desiki-high changed the title fix: filter platforms when metric ImageSize fix: filter platforms when set metric ImageSize May 12, 2023
Signed-off-by: Yadong Ding <ding_yadong@foxmail.com>
Copy link
Collaborator

@imeoer imeoer left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@imeoer imeoer merged commit 4264cf2 into goharbor:main May 12, 2023
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.

2 participants