Skip to content

Add S3VectorsDeleteIndexOperator#66361

Merged
vincbeck merged 1 commit intoapache:mainfrom
john-jac:add-s3vectors-delete-index
May 4, 2026
Merged

Add S3VectorsDeleteIndexOperator#66361
vincbeck merged 1 commit intoapache:mainfrom
john-jac:add-s3vectors-delete-index

Conversation

@john-jac
Copy link
Copy Markdown
Contributor

@john-jac john-jac commented May 4, 2026

What

Add S3VectorsDeleteIndexOperator to delete indexes from Amazon S3 Vectors vector buckets.

Why

Complements S3VectorsCreateIndexOperator to provide full index lifecycle management. The system test previously used a @task with raw boto3 for deletion.

What was done

  • Operator: S3VectorsDeleteIndexOperator added to existing s3_vectors.py
  • Unit tests: Added to existing test_s3_vectors.py — happy path, template fields
  • System test: Updated example_s3_vectors.py — replaced @task delete_index with operator
  • Docs: Updated s3_vectors.rst with new section

@john-jac john-jac requested a review from o-nikolas as a code owner May 4, 2026 15:09
@boring-cyborg boring-cyborg Bot added area:dev-tools area:providers backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch kind:documentation provider:amazon AWS/Amazon - related issues labels May 4, 2026
Copy link
Copy Markdown
Contributor

@vincbeck vincbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check which files are being committed before creating the PR

@john-jac john-jac force-pushed the add-s3vectors-delete-index branch from 7c2d668 to 54f4465 Compare May 4, 2026 17:33
@john-jac
Copy link
Copy Markdown
Contributor Author

john-jac commented May 4, 2026

Please check which files are being committed before creating the PR

Not sure why it was being picked up all of a sudden, but it's fixed now

@john-jac john-jac force-pushed the add-s3vectors-delete-index branch from 54f4465 to 675fabe Compare May 4, 2026 18:21
@john-jac john-jac force-pushed the add-s3vectors-delete-index branch from 675fabe to df84bd7 Compare May 4, 2026 19:08
@vincbeck vincbeck removed the backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch label May 4, 2026
@vincbeck vincbeck merged commit 1662068 into apache:main May 4, 2026
94 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants