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

Define common API for querying registries

Open mthalman opened this issue 1 year ago • 3 comments

This provides a common API, IRegistryContentClient, that provides an interface for querying container registries. This provides a common interface for both the ACR that we own as well as 3rd party registries. For our ACR, it uses the Azure library which provides the Azure auth implementation we require. The logic for determining which implementation to use is in RegistryClientFactory. For now, this uses a hardcoded registry name to determine which implementation it should use. This is temporary. Once I get signoff on these changes, I'll replace that so that the value can be passed in via the command call. That will modify a whole bunch of files so I want to wait until the end to make that change.

mthalman avatar Apr 29 '24 21:04 mthalman

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

I was able to revert all the changes which flowed RegistryAuthContext all over the place and replaced it with a DI solution: https://github.com/dotnet/docker-tools/pull/1262/commits/02e1e8c3ed34f200488474dade29bc90491862ad.

mthalman avatar May 01 '24 19:05 mthalman

I'm going to hold off on merging this until https://github.com/dotnet/docker-tools/pull/1260 is merged and rolled out.

mthalman avatar May 01 '24 21:05 mthalman