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.

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

fix: registry missing layer when concurrently push image #2404

Merged
merged 1 commit into from
Jul 31, 2023
Merged

fix: registry missing layer when concurrently push image #2404

merged 1 commit into from
Jul 31, 2023

Conversation

zhenxuanxin
Copy link

@zhenxuanxin zhenxuanxin commented Jul 31, 2023 •
edited
Loading

this PR fix the following issues:

  1. MANIFEST_BLOB_UNKNOWN occur in registry when push image that most layer are same;
  2. image repository not create in registry when push image that all layer are same.

@zhenxuanxin zhenxuanxin force-pushed the fix-tracker branch 2 times, most recently from 5b2206c to 6bcbb55 Compare July 31, 2023 03:12
@zhenxuanxin
fix: registry missing layer when concurrently push image
5793cca 
Signed-off-by: Xuanxin Zhen <zhenxuanxin@virtaitech.com>
@zhenxuanxin zhenxuanxin marked this pull request as ready for review July 31, 2023 06:05
AkihiroSuda
AkihiroSuda approved these changes Jul 31, 2023
View reviewed changes
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda added this to the v1.5.0 milestone Jul 31, 2023
@AkihiroSuda AkihiroSuda merged commit b33a58f into containerd:main Jul 31, 2023
21 checks passed
@zhenxuanxin zhenxuanxin deleted the fix-tracker branch July 31, 2023 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AkihiroSuda AkihiroSuda AkihiroSuda approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@zhenxuanxin @AkihiroSuda