Implement Azure Container Registry for Container images

[jaredfholgate]

Overview/Summary

This PR replaces the docker hub based image with a local ACR that builds the image on demand.

This was a long planned addition, but has been expedited by throttling that has been applied to Docker Hub.

We were considering a move to MAR / MCR, but this is too challenging.

This PR fixes/adds/changes/removes

1. Feature Request: Migrate container images Azure-Landing-Zones#2529

Breaking Changes

1. Containers are now hosted in ACR

Testing Evidence

e2e tests have been run that cover all the relevant use cases for this change.

As part of this Pull Request I have

• Checked for duplicate Pull Requests
• Associated it with relevant issues, for tracking and closure.
• Ensured my code/branch is up-to-date with the latest changes in the main branch
• Performed testing and provided evidence.
• Updated relevant and associated documentation.

[luke-taylor]

@jaredfholgate Is it worth wrapping private dns zones/private endpoints into a for_each now if we have multiple instances?

[jaredfholgate]

@jaredfholgate Is it worth wrapping private dns zones/private endpoints into a for_each now if we have multiple instances?

Good suggestion, have made that change. I appreciate the code could be neater in general. My intention is to do more refactoring when time allows and move as much as possible to AVM too.

[matt-FFFFFF]

Sorry slipped though. LGTM!