Skip to content

feat(oss): replace custom resource with l1 cfnindex #1056

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

krokoko
Copy link
Collaborator

@krokoko krokoko commented Apr 2, 2025

Fixes #1054

  • Remove custom resource and Docker dependency for OSS Index
  • Use instead the newly released OSS CfnIndex L1 construct

Current status:

  • Need to finish implementation (add the fromVectorIndexAttributes and fromCfnIndex methods)
  • Review interfaces and include helper methods if needed
  • Finish documentation
  • Test !

Blockers:
The newly released L1 index prevents feature parity with existing implementation:


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Copy link
Contributor

This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

@github-actions github-actions bot added the stale label Apr 22, 2025
Copy link
Contributor

Closing this pull request as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

@github-actions github-actions bot closed this Apr 27, 2025
@krokoko krokoko reopened this Apr 27, 2025
Copy link
Contributor

Closing this pull request as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

@github-actions github-actions bot closed this Apr 30, 2025
@krokoko krokoko reopened this Apr 30, 2025
Signed-off-by: Alain Krok <alkrok@amazon.com>
Copy link
Contributor

github-actions bot commented May 4, 2025

Closing this pull request as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

@github-actions github-actions bot closed this May 4, 2025
@krokoko krokoko reopened this May 5, 2025
@krokoko krokoko added backlog and removed stale labels May 5, 2025
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.

(opensearch serverless): use L1 OSS index instead of custom resource
2 participants