Skip to content

[azure-cli-extensions] Remove lambda-test source after deprecation - #10328

Open
Boogie Man (AlexiusTatius-X) wants to merge 2 commits into
Azure:mainfrom
AlexiusTatius-X:remove-lambda-test-source
Open

[azure-cli-extensions] Remove lambda-test source after deprecation#10328
Boogie Man (AlexiusTatius-X) wants to merge 2 commits into
Azure:mainfrom
AlexiusTatius-X:remove-lambda-test-source

Conversation

@AlexiusTatius-X

@AlexiusTatius-X Boogie Man (AlexiusTatius-X) commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

🤖 PR Validation — ️✔️ All clear

Breaking Changes
️✔️ None

Summary

Removes the deprecated lambda-test Azure CLI extension source and its remaining command metadata after the extension index entry was removed and merged in #10270.

Changes

  • Deleted src/lambda-test/ (28 files).
  • Removed the stale az lambda-test service mapping from src/service_name.json.
  • No other extensions were changed.
  • Published wheels are not recalled; this is source cleanup after deprecation, not a new extension release.

Validation

  • python scripts/ci/test_index.py -q: 7 passed, 2 skipped.
  • Scope audit: 28 source deletions plus one src/service_name.json metadata edit; no unexpected paths.
  • src/index.json and src/service_name.json contain no lambda-test registration.
  • src/service_name.json parses as valid JSON.
  • git diff --check upstream/main...HEAD: clean.

Release metadata

HISTORY.rst and setup.py are deleted with the retired extension. No version bump is intended because this PR does not publish a new wheel; publishing one would conflict with the completed index deprecation.

Tracking

Copilot AI lite review requested due to automatic review settings September 11, 2026 09:57
@azure-client-tools-bot-prd

Copy link
Copy Markdown

Hi Boogie Man (@AlexiusTatius-X),
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in pyproject.toml (or setup.py, if the extension has not migrated yet) as well.

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.

🔵 Needs a closer look

Remove the stale az lambda-test registry entry from src/service_name.json.

Pull request overview

Removes the deprecated lambda-test Azure CLI extension source.

Changes:

  • Deletes packaging, documentation, metadata, generated commands, tests, and recordings.
  • Requires removal of the stale az lambda-test entry from src/service_name.json.
File summaries
File Change
src/lambda-test/setup.py Deleted packaging configuration
src/lambda-test/setup.cfg Deleted package configuration
src/lambda-test/README.md Deleted documentation
src/lambda-test/HISTORY.rst Deleted release history
src/lambda-test/azext_lambda_test/tests/latest/test_lambda_test.py Deleted scenario test
src/lambda-test/azext_lambda_test/tests/latest/recordings/test_lambda_test.yaml Deleted test recording
src/lambda-test/azext_lambda_test/tests/latest/__init__.py Deleted test package marker
src/lambda-test/azext_lambda_test/tests/__init__.py Deleted test package marker
src/lambda-test/azext_lambda_test/custom.py Deleted custom command hooks
src/lambda-test/azext_lambda_test/commands.py Deleted command loader
src/lambda-test/azext_lambda_test/azext_metadata.json Deleted extension metadata
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/_wait.py Deleted wait command
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/_update.py Deleted update command
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/_show.py Deleted show command
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/_list.py Deleted list command
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/_delete.py Deleted delete command
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/_create.py Deleted create command
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/__init__.py Deleted organization command package
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/organization/__cmd_group.py Deleted organization command group
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/__init__.py Deleted command group package
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/hyper_execute/__cmd_group.py Deleted HyperExecute command group
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/__init__.py Deleted command group package
src/lambda-test/azext_lambda_test/aaz/latest/lambda_test/__cmd_group.py Deleted root command group
src/lambda-test/azext_lambda_test/aaz/latest/__init__.py Deleted generated package marker
src/lambda-test/azext_lambda_test/aaz/__init__.py Deleted AAZ package marker
src/lambda-test/azext_lambda_test/_params.py Deleted parameter definitions
src/lambda-test/azext_lambda_test/_help.py Deleted help definitions
src/lambda-test/azext_lambda_test/__init__.py Deleted extension loader
Review details

Suppressed comments (1)

src/lambda-test/azext_lambda_test/init.py:1

  • Removing the extension package leaves a live registry entry in src/service_name.json (lines 1023-1025) for az lambda-test, even though this source no longer supplies that command. That makes the service-name metadata advertise a nonexistent extension; remove that entry as part of the cleanup (and adjust any generated consumers if required).
  • Files reviewed: 27/28 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@yonzhan

Copy link
Copy Markdown
Collaborator

Remove lambda-test

@AlexiusTatius-X

Copy link
Copy Markdown
Contributor Author

Thanks for the guidance. This PR is post-deprecation source cleanup, not an extension release: the index entry was already removed in #10270, and this PR deletes the retired extension directory, including its HISTORY.rst and setup.py. No new wheel or version bump is intended, since publishing a new version would conflict with the completed index deprecation.

I also removed the remaining az lambda-test mapping from src/service_name.json, following the merged Neon source-removal precedent in #9715. The PR description now records this scope and the release-metadata rationale.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants