Skip to content

Azure Docs: Azure Container Registry#598

Merged
remotesynth merged 2 commits into
mainfrom
docs/azure-container-registry
May 11, 2026
Merged

Azure Docs: Azure Container Registry#598
remotesynth merged 2 commits into
mainfrom
docs/azure-container-registry

Conversation

@paolosalvatori
Copy link
Copy Markdown
Contributor

This pull request modifies the existing article on Azure Container Registry in the Azure emulator documentation.

Fixes DOC-200

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 22, 2026

Deploying localstack-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: e0820a4
Status: ✅  Deploy successful!
Preview URL: https://15b43074.localstack-docs.pages.dev
Branch Preview URL: https://docs-azure-container-registr.localstack-docs.pages.dev

View logs

remotesynth added a commit that referenced this pull request Apr 23, 2026
This merges content from #598 into the original PR.
1. **Tier mismatch:** Body said “Standard-tier” while the command used **`--sku Basic`** → clarified as Basic-tier and pointed to the SKU article.
2. **Location mismatch:** Commands used **`westeurope`** but sample outputs showed **`eastus`** → outputs now use **`westeurope`** so they match the commands.
3. **Wrong subsection description:** “**Enable** the admin user” while the command was **`--admin-enabled false`** → text now describes **disabling** the admin account, with a link to [`az acr update`](https://learn.microsoft.com/en-us/cli/azure/acr#az-acr-update).
4. **`check-name` context:** Clarified that the sample assumes **`myacrdemo` already exists**, so **`AlreadyExists`** is the expected illustration.
5. **Login explanation:** Replaced a non-working Learn URL with [Docker CLI push/pull / sign-in](https://learn.microsoft.com/en-us/azure/container-registry/container-registry-get-started-docker-cli?tabs=azure-cli) and [Authentication overview](https://learn.microsoft.com/en-us/azure/container-registry/container-registry-authentication); kept “emulator may warn then succeed” without changing `azlocal`.
6. **Samples link:** Corrected GitHub path to `https://github.com/localstack/localstack-azure-samples/blob/main/samples/aci-blob-storage/python/README.md` (valid `blob/main/` URL).
Copilot AI review requested due to automatic review settings May 11, 2026 18:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@remotesynth
Copy link
Copy Markdown
Collaborator

Minor accuracy updates:

  1. Tier mismatch: Body said “Standard-tier” while the command used --sku Basic → clarified as Basic-tier and pointed to the SKU article.
  2. Location mismatch: Commands used westeurope but sample outputs showed eastus → outputs now use westeurope so they match the commands.
  3. Wrong subsection description:Enable the admin user” while the command was --admin-enabled false → text now describes disabling the admin account, with a link to az acr update.
  4. check-name context: Clarified that the sample assumes myacrdemo already exists, so AlreadyExists is the expected illustration.
  5. Login explanation: Replaced a non-working Learn URL with Docker CLI push/pull / sign-in and Authentication overview; kept “emulator may warn then succeed” without changing azlocal.
  6. Samples link: Corrected GitHub path to https://github.com/localstack/localstack-azure-samples/blob/main/samples/aci-blob-storage/python/README.md (valid blob/main/ URL).

@remotesynth remotesynth merged commit 4acabb1 into main May 11, 2026
1 check passed
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.

3 participants