Skip to content

[Containerapp] az containerapp env create: Pre-announce future logs destination default change - #34243

Open
Michael Dai (michaelkira) wants to merge 1 commit into
Azure:devfrom
michaelkira:private/michdai/containerapp-logs-preannouncement
Open

Michael Dai (michaelkira) wants to merge 1 commit into
Azure:devfrom
michaelkira:private/michdai/containerapp-logs-preannouncement

Conversation

@michaelkira

@michaelkira Michael Dai (michaelkira) commented Oct 20, 2026 •

Copy link
Copy Markdown

🤖 PR Validation — ️✔️ All clear

Breaking Changes Tests
️✔️ None ️✔️ 130/130

Related command

az containerapp env create

Description

Pre-announces the future change of the --logs-destination default from log-analytics to azure-monitor using the standard breaking-change registration.

This PR does NOT change the current default or create behavior. The warning links to the migration information in #34228. To retain Log Analytics behavior after the eventual change, users should explicitly pass --logs-destination log-analytics.

Release coordination

  • Implementation PR: [Containerapp] BREAKING CHANGE: Change default logs destination to Azure Monitor #34228
  • This announcement must ship in a released CLI version before the minimum 30-day notice period starts. Merging this PR alone does not start that period.
  • Hold the implementation PR until the notice period has elapsed and the CLI team confirms an eligible breaking-change window.
  • The currently listed November 16, 2026 breaking-change release is less than 30 days from October 20, 2026. This PR does not promise that release.
  • Uses target_version=None, following existing VM pre-announcements, to say "in a future release" rather than automatically promising the next window. Maintainers: please confirm the target version/window and announcement release.
  • After the announcement is published, its release version/date and the confirmed implementation target should be recorded in the implementation PR.

Testing Guide

  • python -m pytest src\azure-cli\azure\cli\command_modules\containerapp\tests\latest\test_containerapp_breaking_change.py -q: PASSED (1 test). Verifies the warning registration, always-displayed tag, future-release wording, migration link, and unchanged handler default.
  • azdev style azure-cli-containerapp: PASSED (Pylint and Flake8).
  • Mocked CLI invocation of containerapp env create -g test-rg -n test-env -l australiaeast: PASSED. Confirmed the runtime warning is emitted and the handler receives log-analytics; the creation decorator was mocked and extension discovery disabled for isolation.

History Notes

[Containerapp] az containerapp env create: Pre-announce the future change of the --logs-destination default from log-analytics to azure-monitor; the current default remains unchanged


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

Keep the default at log-analytics and warn about the future azure-monitor default without promising an unconfirmed release window.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 20, 2026 04:53
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@microsoft-github-policy-service

Copy link
Copy Markdown
Contributor

Thank you for your contribution Michael Dai (@michaelkira)! We will review the pull request and get back to you soon.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The scoped changes use existing warning infrastructure, preserve current behavior, and have no identified blocking issues.

No open issues 🎉

What changed in this PR

Pre-announces the future Container Apps logs destination default change without changing current creation behavior.

Changes:

  • Registers a future-release warning with a migration link, without promising a release window.
  • Adds a regression test for the warning and unchanged log-analytics default.
File Description
src/​azure-cli/​azure/​cli/​command_modules/​containerapp/​tests/​latest/​test_containerapp_breaking_change.py Verifies warning registration, wording, visibility, and current default.
src/​azure-cli/​azure/​cli/​command_modules/​containerapp/​_breaking_change.py Registers the default-change pre-announcement.

Review effort: Balanced

@a0x1ab

Copy link
Copy Markdown
Member

/azp run

@x-engineering-agent

Copy link
Copy Markdown
Contributor

Hi Michael Dai (@michaelkira),
Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants