-
Notifications
You must be signed in to change notification settings - Fork 205
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
[backport v2.2] nydusify: support --with-referrer option #1389
[backport v2.2] nydusify: support --with-referrer option #1389
Conversation
@adamqqqplay , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/88745 |
@adamqqqplay , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
Signed-off-by: Yan Song <[email protected]> Signed-off-by: Qinqi Qu <[email protected]>
With this option, we can track all nydus images associated with an OCI image. For example, in Harbor we can cascade to show nydus images linked to an OCI image, deleting the OCI image can also delete the corresponding nydus images. At runtime, nydus snapshotter can also automatically upgrade an OCI image run to nydus image. Prior to this PR, we had enabled this feature by default. However, it is now known that Docker Hub does not yet support Referrer. Therefore, adding this option to disable this feature by default, to ensure broad compatibility with various image registries. Fix dragonflyoss#1363. Signed-off-by: Yan Song <[email protected]>
da7069d
to
94f42fe
Compare
@adamqqqplay , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/88760 |
@adamqqqplay , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
/retest |
@adamqqqplay , the test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/88761 |
@adamqqqplay , The CI test is completed, please check result:
Congratulations, your test job passed! |
@adamqqqplay Thanks for the backport! |
Relevant Issue (if applicable)
#1363 #1384
Details
With this option, we can track all nydus images associated with
an OCI image. For example, in Harbor we can cascade to show nydus
images linked to an OCI image, deleting the OCI image can also delete
the corresponding nydus images. At runtime, nydus snapshotter can also
automatically upgrade an OCI image run to nydus image.
Prior to this PR, we had enabled this feature by default. However,
it is now known that Docker Hub does not yet support Referrer.
Therefore, adding this option to disable this feature by default,
to ensure broad compatibility with various image registries.
Fix #1363.
Types of changes
What types of changes does your PullRequest introduce? Put an
x
in all the boxes that apply:Checklist
Go over all the following points, and put an
x
in all the boxes that apply.