dotnet-docker icon indicating copy to clipboard operation
dotnet-docker copied to clipboard

Remove `core` syndication

Open richlander opened this issue 2 years ago • 4 comments

Do we still need this support?

https://github.com/dotnet/dotnet-docker/blob/f4786b8c0b4469f7eb18f891fd6c090561e50006/manifest.samples.json#L18-L31

https://github.com/dotnet/dotnet-docker/blob/f4786b8c0b4469f7eb18f891fd6c090561e50006/manifest.json#L7-L12

richlander avatar Mar 31 '23 05:03 richlander

This should be resolved before https://github.com/microsoft/mcr/pull/3010 can be merged.

It's also necessary to have this done as a prerequisite to the EOL annotation work since these are still supported tags at this point.

mthalman avatar Aug 20 '24 19:08 mthalman

[Triage] Related: https://github.com/dotnet/docker-tools/issues/1201

The unused variables from dotnet-docker's manifest can be removed today.

There is an open question about when we can safely remove the active syndication for dotnet/samples repo. There are a few things we can check:

  • Telemetry to see if any dotnet/core/samples images are being pulled
  • Search across Github / AzDO repos to see if there are any significant usages of these images

If we don't determine there to be any significant use of the dotnet/core/samples images, then we should be OK to stop syndicating them. If there is a non-trivial amount of usage then we'll need an announcement.

lbussell avatar Aug 26 '24 18:08 lbussell

Doing a search on GitHub, I'm still seeing quite a bit of usage of mcr.microsoft.com/dotnet/core/samples: https://github.com/search?q=mcr.microsoft.com%2Fdotnet%2Fcore%2Fsamples+-is%3Afork&type=code&ref=advsearch. That search is currently returning 235 files.

mthalman avatar Aug 27 '24 15:08 mthalman

Announcement has been posted at https://github.com/dotnet/dotnet-docker/discussions/5869

mthalman avatar Sep 10 '24 17:09 mthalman

EOL annotations have been applied to all the images in the core and core-nightly repos. Last step is to unregister those repos from MAR: https://github.com/microsoft/mcr/pull/3010.

mthalman avatar Nov 13 '24 17:11 mthalman

Fixed by https://github.com/dotnet/dotnet-docker/pull/6047.

mthalman avatar Nov 13 '24 17:11 mthalman