{Pinecone} Remove leftover pinecone extension source (RP deprecated) - #10159
Open
AlexiusTatius-X wants to merge 1 commit into
Open
{Pinecone} Remove leftover pinecone extension source (RP deprecated)#10159AlexiusTatius-X wants to merge 1 commit into
AlexiusTatius-X wants to merge 1 commit into
Conversation
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the residual (unpublished) pinecone Azure CLI extension source under src/pinecone/, aligning the repo with the prior removal of pinecone entries from the extension index/metadata and the resource provider’s deprecation.
Changes:
- Deletes the full
src/pinecone/extension source tree (packaging, help/params/commands, AAZ command implementation, and tests/recordings). - Removes the extension’s setup and documentation files (
setup.py,README.md,HISTORY.rst, etc.). - Removes the extension’s scenario test and its recordings.
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/pinecone/setup.py | Removed deprecated extension packaging script. |
| src/pinecone/setup.cfg | Removed deprecated extension packaging config. |
| src/pinecone/README.md | Removed deprecated extension README. |
| src/pinecone/HISTORY.rst | Removed deprecated extension changelog. |
| src/pinecone/azext_pinecone/init.py | Removed extension command loader entrypoint. |
| src/pinecone/azext_pinecone/_help.py | Removed extension help definitions module. |
| src/pinecone/azext_pinecone/_params.py | Removed extension parameter registration module. |
| src/pinecone/azext_pinecone/commands.py | Removed extension command table module. |
| src/pinecone/azext_pinecone/custom.py | Removed extension custom commands module. |
| src/pinecone/azext_pinecone/azext_metadata.json | Removed extension metadata file. |
| src/pinecone/azext_pinecone/aaz/init.py | Removed AAZ package init for the extension. |
| src/pinecone/azext_pinecone/aaz/latest/init.py | Removed AAZ “latest” package init. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/init.py | Removed AAZ command group package init for pinecone. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/__cmd_group.py | Removed AAZ command group registration for pinecone. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/vector_db/init.py | Removed AAZ command group package init for vector-db. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/vector_db/__cmd_group.py | Removed AAZ command group registration for vector-db. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/vector_db/organization/init.py | Removed AAZ command group package init for organization. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/vector_db/organization/__cmd_group.py | Removed AAZ command group registration for organization. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/vector_db/organization/_list.py | Removed AAZ implementation for organization list. |
| src/pinecone/azext_pinecone/aaz/latest/pinecone/vector_db/organization/_show.py | Removed AAZ implementation for organization show. |
| src/pinecone/azext_pinecone/tests/init.py | Removed extension tests package init. |
| src/pinecone/azext_pinecone/tests/latest/init.py | Removed extension “latest” tests package init. |
| src/pinecone/azext_pinecone/tests/latest/test_pinecone.py | Removed scenario test for the deprecated extension. |
| src/pinecone/azext_pinecone/tests/latest/recordings/test_pinecone_all.yaml | Removed test recordings tied to the deprecated extension. |
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
Collaborator
|
Pinecone |
Contributor
Author
|
Hi, is there anything else that is needed to be reviewed ? |
Collaborator
|
@necusjz could help review |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 PR Validation — ️✔️ All clear
Summary
Removes the leftover
src/pineconeextension source. Thepinecone(Microsoft.Pinecone.VectorDb) CLI extension was pulled pre-publish in May 2025 — itssrc/index.jsonentry was removed in #8763 and itssrc/service_name.jsonentry in #8768 — but the source folder undersrc/pineconewas never deleted. TheMicrosoft.Pinecone.VectorDbresource provider is now being fully deprecated, so this removes the residual source.What this PR does
src/pinecone/(24 files) — the entire unpublished/deprecated extension.Notes
src/index.json—pineconeis already absent (removed in {Pinecone} Remove liftr pinecone as service not public #8763), so the extension is not installable and no user is affected.src/service_name.json— already absent (removed in {doc} Remove unpublished modulepineconeandweights-and-biasesentry inservice_name.json#8768).This checklist is used to make sure that common guidelines for a pull request are followed.