Skip to content

[20.10 backport] registry: allow "allow-nondistributable-artifacts" for Docker Hub#44313

Merged
thaJeztah merged 1 commit into
moby:20.10from
tianon:20.10-distributable
Oct 19, 2022
Merged

[20.10 backport] registry: allow "allow-nondistributable-artifacts" for Docker Hub#44313
thaJeztah merged 1 commit into
moby:20.10from
tianon:20.10-distributable

Conversation

@tianon

@tianon tianon commented Oct 17, 2022

Copy link
Copy Markdown
Member

Cherry-pick of #44305

Previously, Docker Hub was excluded when configuring "allow-nondistributable-artifacts". With the updated policy announced by Microsoft, we can remove this restriction; https://techcommunity.microsoft.com/t5/containers/announcing-windows-container-base-image-redistribution-rights/ba-p/3645201

There are plans to deprecated support for foreign layers altogether in the OCI, and we should consider to make this option the default, but as that requires deprecating the option (and possibly keeping an "opt-out" option), we can look at that separately.

(cherry picked from commit 30e5333)

Previously, Docker Hub was excluded when configuring "allow-nondistributable-artifacts".
With the updated policy announced by Microsoft, we can remove this restriction;
https://techcommunity.microsoft.com/t5/containers/announcing-windows-container-base-image-redistribution-rights/ba-p/3645201

There are plans to deprecated support for foreign layers altogether in the OCI,
and we should consider to make this option the default, but as that requires
deprecating the option (and possibly keeping an "opt-out" option), we can look
at that separately.

(cherry picked from commit 30e5333)

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Tianon Gravi <[email protected]>
@tianon
tianon force-pushed the 20.10-distributable branch from 47e6bf8 to 6699afa Compare October 17, 2022 22:12
@thaJeztah
thaJeztah marked this pull request as draft October 17, 2022 22:19

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

moved it to draft as some other changes have to go in first.

Could you open a backport for 22.06 as well?

@thaJeztah

Copy link
Copy Markdown
Member

ah! Found the 22.06 one; thanks!

@thaJeztah thaJeztah added this to the 20.10.20 milestone Oct 17, 2022
@thaJeztah
thaJeztah marked this pull request as ready for review October 18, 2022 19:51
@thaJeztah thaJeztah modified the milestones: 20.10.20, 20.10.21 Oct 18, 2022
@tianon

tianon commented Oct 19, 2022

Copy link
Copy Markdown
Member Author

This one is OK now? 😇

@thaJeztah

Copy link
Copy Markdown
Member

I think it is 👍

@thaJeztah
thaJeztah merged commit 2b7f97a into moby:20.10 Oct 19, 2022
@tianon
tianon deleted the 20.10-distributable branch October 19, 2022 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants