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.

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

Define version of Image Builder image with Azure CLI installed #1068

Merged
merged 2 commits into from
Oct 26, 2022

Conversation

mthalman
Copy link
Member

In order to implement dotnet/dotnet-docker#4139, we need the ability to authenticate to the internal ACR. This requires the use of the Azure CLI to be able to authenticate into Azure with a certificate.

Installing the Azure CLI adds 1 GB to the image size, a 270% increase over the original image size of Image Builder. The large install size is already tracked by Azure/azure-cli#7387. Given that the use of the Azure CLI is only needed in scenarios to target internal builds, it doesn't seem like a good idea to have it installed for the Image Builder image used everywhere.

With these changes, I've defined a separate image that adds the Azure CLI to the base Image Builder image. This will allow the pipeline runs that target public builds to not be subjected to the very large image size. It will only be relevant for runs that target internal builds.

@dotnet-issue-labeler
Copy link

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.

@mthalman mthalman marked this pull request as draft October 18, 2022 18:52
@mthalman
Copy link
Member Author

Converted to draft. Need to make some more changes here.

@mthalman mthalman removed the request for review from MichaelSimons October 18, 2022 18:53
@mthalman mthalman marked this pull request as ready for review October 24, 2022 15:49
@mthalman mthalman merged commit ed310cb into dotnet:main Oct 26, 2022
@mthalman mthalman deleted the az-cli branch October 26, 2022 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants