[ContainerApp] Fix az containerapp up --source crash: resolve OS/Architecture models from correct SDK package

[lygav]

Related command
az containerapp up --source

Description

az containerapp up --source . crashes with AttributeError: 'NoneType' object has no attribute 'linux' because queue_acr_build() in _utils.py resolves OS, Architecture, DockerBuildRequest, and PlatformProperties models via cmd.get_models() using ResourceType.MGMT_CONTAINERREGISTRY with operation_group='runs'.

These models have moved to the azure-mgmt-containerregistrytasks SDK package, so cmd.get_models() silently returns None, and the subsequent OS.linux.value call crashes.

This PR updates both get_models() calls in queue_acr_build() to use ResourceType.MGMT_CONTAINERREGISTRYTASKS, which is already a declared dependency and contains the correct models.

Testing Guide

# Create a directory with a Dockerfile
mkdir test-app && cd test-app
cat > Dockerfile <<EOF
FROM ubuntu:24.04
CMD ["sleep", "infinity"]
EOF

# This should no longer crash
az containerapp up -n my-app -g my-rg --environment my-env --source .

The existing @live_only tests in test_containerapp_compose_registries.py cover queue_acr_build and would validate this fix in the nightly live test run.

History Notes

[ContainerApp] Fix az containerapp up --source: resolve OS/Architecture models from correct SDK package (MGMT_CONTAINERREGISTRYTASKS)

Fixes #33369

---

This checklist is used to make sure that common guidelines for a pull request are followed.

• The PR title and description has followed the guideline in Submitting Pull Requests.

• I adhere to the Command Guidelines.

• I adhere to the Error Handling Guidelines.

[azure-client-tools-bot-prd]

Validation for Azure CLI Full Test Starting...

Thanks for your contribution!

[azure-client-tools-bot-prd]

Validation for Breaking Change Starting...

Thanks for your contribution!

[yonzhan]

Thank you for your contribution! We will review the pull request and get back to you soon.

[github-actions]

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

[microsoft-github-policy-service]

Thank you for your contribution @lygav! We will review the pull request and get back to you soon.

[Copilot]

Pull request overview

Fixes a crash in az containerapp up --source during ACR build queuing by resolving OS, Architecture, DockerBuildRequest, and PlatformProperties from the correct SDK (azure-mgmt-containerregistrytasks) via ResourceType.MGMT_CONTAINERREGISTRYTASKS, avoiding cmd.get_models() returning None.

Changes:

• Update queue_acr_build() to load OS/Architecture models from MGMT_CONTAINERREGISTRYTASKS instead of MGMT_CONTAINERREGISTRY.
• Update queue_acr_build() to load DockerBuildRequest/PlatformProperties models from MGMT_CONTAINERREGISTRYTASKS instead of MGMT_CONTAINERREGISTRY.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.