Repository navigation
[Containerapp] BREAKING CHANGE: Change default logs destination to Azure Monitor - #34228
Open
Michael Dai (michaelkira) wants to merge 2 commits into
Open
Michael Dai (michaelkira) wants to merge 2 commits into
Michael Dai (michaelkira) wants to merge 2 commits into
Conversation
…ure Monitor Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Michael Dai (michaelkira)
requested review from
a team,
Greedygre (Greedygre),
Zunli Hu (Juliehzl) and
Hong Wang (howang-ms)
as code owners
October 15, 2026 06:38
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
microsoft-github-policy-service
Bot
requested review from
ZelinWang (wangzelin007) and
Yong Zhang (yonzhan)
October 15, 2026 06:39
Contributor
|
Thank you for your contribution Michael Dai (@michaelkira)! We will review the pull request and get back to you soon. |
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The shared default causes containerapp up to silently ignore explicit Log Analytics workspace settings when creating an environment.
3 open findings
What changed in this PR
Changes Container Apps environment creation to use Azure Monitor by default instead of Log Analytics.
Changes:
- Updates the shared creation handler’s
logs_destinationdefault to"azure-monitor". - Leaves explicitly supplied destination values unchanged.
| File | Description |
|---|---|
| src/azure-cli/azure/cli/command_modules/containerapp/custom.py | Changes the default logging destination for managed environment creation. |
🧠 Review effort: Balanced
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


🤖 PR Validation — ❌ Action needed
❌AzureCLI-BreakingChangeTest
Please submit your Breaking Change Pre-announcement ASAP if you haven't already. Please note:
Related command
az containerapp env createDescription
Changes the default value of
--logs-destinationfromlog-analyticstoazure-monitorwhen creating a Container Apps managed environment. Explicit--logs-destinationvalues continue to work unchanged.This is a breaking change because callers that omit
--logs-destinationwill now create environments configured for Azure Monitor instead of Log Analytics.Testing Guide
azdev style azure-cli-containerapp- Passed (Pylint and Flake8).azdev test containerapp.test_containerapp_env_default_logs_destination --profile latest --series --live --discover- Passed and generated the recording.azdev test containerapp.test_containerapp_env_default_logs_destination --profile latest --series- Passed in playback mode.History Notes
[Containerapp] BREAKING CHANGE:
az containerapp env create: Change the default value of--logs-destinationfromlog-analyticstoazure-monitorThis 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.