Update test utility to remove default resource group - #33853
Update test utility to remove default resource group#33853Matthew Metcalf (mrm9084) wants to merge 1 commit into
Conversation
Remove default resource group value for tests.
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
Pull request overview
This PR updates the App Configuration test utilities to stop using a hard-coded default resource group name, relying instead on an environment variable to supply the resource group for data-plane tests.
Changes:
- Removed the default fallback value from
get_test_resource_group()so it only readsAZURE_CLI_APPCONFIG_TEST_RG.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Live test results —
|
|
Matthew Metcalf (@mrm9084) Please fix the breaking tests for appconfig as outlined in the PR Validation summary |
There was a problem hiding this comment.
CI Summary
This PR currently has failing checks that need attention before it can be merged:
- ❌ Live test (azdev test,
--live) — the dispatched live-test workflow finished with a failure: https://github.com/Azure/issue-sentinel/actions/runs/32222736654 - ❌ Azure.azure-cli (Build) — https://github.com/Azure/azure-cli/runs/92386122903
- ❌ Azure.azure-cli Full Test — https://github.com/Azure/azure-cli/runs/92391032291
- ❌ Automation Full Test Python312 Profile Latest instance5 — https://github.com/Azure/azure-cli/runs/92380288948
- ❌ Automation Full Test Python314 Profile Latest instance5 — https://github.com/Azure/azure-cli/runs/92378350491
- ❌ Check the Format of Pull Request Title and Content — https://github.com/Azure/azure-cli/runs/92372713681
Please review the linked check logs, fix the failing tests/build, and update the PR title/description to match the required format (see the PR template and title guidelines). Once fixed, CI will re-run automatically.
|
This PR has been replaced by #33906 |
🤖 PR Validation — ❌ Action needed
❌AzureCLI-FullTest
Remove default resource group value for tests.
Related command
Description
Testing Guide
History Notes
[Component Name 1] BREAKING CHANGE:
az command a: Make some customer-facing breaking change[Component Name 2]
az command b: Add some customer-facing featureThis 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.