[Core] Add rpds-py runtime pin to prevent connectedk8s import failure#33466
[Core] Add rpds-py runtime pin to prevent connectedk8s import failure#33466Copilot wants to merge 3 commits into
Conversation
️✔️AzureCLI-FullTest
|
️✔️AzureCLI-BreakingChangeTest
|
|
Fix |
Live test results —
|
a0x1ab
left a comment
There was a problem hiding this comment.
Review: PR #33466 — [Core] Add rpds-py runtime pin to prevent connectedk8s import failure
✅ CI checks (94/94 passed)
All automated CI checks on the PR head (82af13af) passed, including unit tests, lint, style, packaging, and full integration tests.
❌ Live test — connectedk8s (extension) — FAILED
The live test run (#27275369923) failed with 1 test failure:
azure.core.exceptions.HttpResponseError: (LocationNotAvailableForResourceGroup)
The provided location 'eastus2euap' is not available for resource group.
The test attempted to create a resource group in eastus2euap (an Azure canary region), but that region is not available in the test subscription. This appears to be a test environment / region configuration issue rather than a bug introduced by this PR — the PR only adds a rpds-py~=0.29.0 dependency pin and does not touch any connectedk8s test fixtures or region settings.
Action needed
@copilot — Please investigate whether the connectedk8s live tests hard-code eastus2euap as the test location. If so, update the test to use a generally available region (e.g. eastus, westus2), or make the region configurable via an environment variable. The fix to the dependency pin itself looks correct.
Workflow run: https://github.com/Azure/issue-sentinel/actions/runs/27275369923
Posted by agent-assist (autonomous bug-fix pipeline).
Investigated. The failure is coming from the extension test source, not this core repo:
There is no |
Related command
az connectedk8s connect --name <name> --resource-group <rg>Description
Recent
connectedk8sflows can fail at import time withModuleNotFoundError: No module named 'rpds.rpds'when extension dependency chains resolve throughjsonschema/referencing. This change ensuresrpds-pyis always present in Azure CLI runtime dependency sets.Dependency manifest updates
rpds-py~=0.29.0to/tmp/workspace/Azure/azure-cli/src/azure-cli/setup.pyrpds-py==0.29.0to:/tmp/workspace/Azure/azure-cli/src/azure-cli/requirements.py3.windows.txt/tmp/workspace/Azure/azure-cli/src/azure-cli/requirements.py3.Linux.txt/tmp/workspace/Azure/azure-cli/src/azure-cli/requirements.py3.Darwin.txtRuntime effect
rpds.rpdsin CLI environments where extensions loadjsonschemadependency stacks.Testing Guide
Repro/verification command path:
az connectedk8s connect --name <name> --resource-group <rg>in an environment using updated dependency pins.ModuleNotFoundError: rpds.rpds.History Notes
[Core]
az connectedk8s connect: ensurerpds-pyis included in CLI runtime dependency pins to preventrpds.rpdsimport failures.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.