Skip to content

[Config] Fix #34186: az config: Remove the blanket experimental warning - #34191

Open
Aditya Pujara (a0x1ab) wants to merge 4 commits into
Azure:devfrom
a0x1ab:agent-assist/azure-azure-cli-issue-34186-f920502ff5cf
Open

Aditya Pujara (a0x1ab) wants to merge 4 commits into
Azure:devfrom
a0x1ab:agent-assist/azure-azure-cli-issue-34186-f920502ff5cf

Conversation

@a0x1ab

Copy link
Copy Markdown
Member

Related command

az config

Description

Fixes #34186.

Remove the blanket experimental warning.

Testing Guide

  • Run az config.
  • Focused regression coverage: src/azure-cli/azure/cli/command_modules/config/tests/latest/test_config.py.
  • Repository CI and the authorized live-test workflow validate the published revision.

History Notes

[Config] az config: Remove the blanket experimental warning


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

@yonzhan

Copy link
Copy Markdown
Collaborator

az config

@necusjz

Copy link
Copy Markdown
Member

@Azure/x-engineering-agent plz,

  1. remove the test case in src/azure-cli/azure/cli/command_modules/config/tests/latest/test_config.py, it's already covered by another framework.
  2. run azdev latest-index generate, it will refresh src/azure-cli-core/azure/cli/core/helpIndex.latest.json.

@a0x1ab

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

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

@yonzhan

Copy link
Copy Markdown
Collaborator

Please fix CI issues

@a0x1ab

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

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

x-engineering-agent[bot]

This comment was marked as outdated.

@x-engineering-agent x-engineering-agent Bot added the X Engineering Agent Reviewed Pull request reviewed by X Engineering Agent label Oct 8, 2026
@a0x1ab
Aditya Pujara (a0x1ab) force-pushed the agent-assist/azure-azure-cli-issue-34186-f920502ff5cf branch from 7c4a49d to 3aa869e Compare October 8, 2026 12:19
@a0x1ab

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

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

@x-engineering-agent
x-engineering-agent Bot marked this pull request as ready for review October 8, 2026 13:58
@x-engineering-agent
x-engineering-agent Bot requested review from a team as code owners October 8, 2026 13:58
Copilot AI balanced review requested due to automatic review settings October 8, 2026 13:58
@azure-pipelines

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

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 focused change matches the issue and includes comprehensive regression coverage.

0 open findings

What changed in this PR

Removes the misleading experimental status from the stable az config command group while retaining it for config param-persist.

Changes:

  • Removes the root command group’s experimental flag.
  • Updates packaged help metadata.
  • Adds regression coverage for commands, errors, local configuration, and parameter persistence.
File Description
commands.py Marks the root config group as stable.
test_config.py Verifies warning behavior and command functionality.
helpIndex.latest.json Removes the experimental help tag.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

@a0x1ab
Aditya Pujara (a0x1ab) force-pushed the agent-assist/azure-azure-cli-issue-34186-f920502ff5cf branch from 3aa869e to 4c244f7 Compare October 8, 2026 15:20
x-engineering-agent[bot]

This comment was marked as outdated.

x-engineering-agent[bot]

This comment was marked as outdated.

@x-engineering-agent x-engineering-agent Bot added Request X Engineering Agent Request X Engineering Agent testing and review and removed Request X Engineering Agent Request X Engineering Agent testing and review labels Oct 9, 2026
x-engineering-agent[bot]

This comment was marked as outdated.

@x-engineering-agent

Copy link
Copy Markdown
Contributor

Live test skipped

⏭️ Skipping the live test for this revision because no changed test file was found (tests/**/test_*.py).

The live-test pipeline runs only the test files a PR changes, so there is nothing to execute for this commit. A skipped live test is not a passing test result. The Agent review separately checks whether the affected command module includes focused regression tests or updated recordings. If a test file is changed in a later commit, the live test will run automatically.

Posted by x-engineering-agent (Tester)

x-engineering-agent[bot]

This comment was marked as outdated.

@x-engineering-agent x-engineering-agent Bot removed the Request X Engineering Agent Request X Engineering Agent testing and review label Oct 9, 2026

@x-engineering-agent x-engineering-agent Bot 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.

Aditya Pujara (@a0x1ab)

CI recovered; current diff is empty

Upstream CI recovered after the previous failed-CI review: 51/51 checks passed, with zero failed, cancelled, or pending checks; latest completion 2026-10-09T09:09:40Z.

Head 489f4ccfa33f262a708a4e287bb6f4b385570f83 currently has no file delta against its base. All seven review tools were accounted for and are not applicable to this empty diff. No changed-line defect or deterministic policy finding is supported. The reused live-test result is a neutral skip, not a passing run.

Please confirm whether this zero-diff PR is superseded. Any validation claim for issue #34186 should include authorized upstream verification that both broker-setting values produce no blanket warning and az config --help has no experimental group notice. Existing assertions and artifact contents alone do not prove that runtime outcome or the approved regeneration requested in human feedback.

Test validation

  • Live test: Skipped: no runnable changed test files were found.
  • Regression coverage: Not applicable to the changed files.

Risk assessment

1/100 · Low · Low confidence

The Low rating reflects the detected change scope with no elevated security, reliability, customer, operational, dependency, sovereign-cloud, generated-output, or cross-component signal.

  • Change scope: 0 changed files, 0 changed lines (+0 / -0), including 0 production files.
  • Affected components: No production component was identified.
  • Risk drivers: No elevated risk signal was detected.
  • Regression evidence: No production-code regression-test signal applies.
  • Confidence: Low because no production changed-line evidence was available.
  • Required review: No additional owning-squad review signal was detected.

Posted by x-engineering-agent (Reviewer)

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

X Engineering Agent Reviewed Pull request reviewed by X Engineering Agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] Remove confusing "Command group 'config' is experimental" warning

4 participants