-
Notifications
You must be signed in to change notification settings - Fork 56
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
feat: implement opensearch serverless support #212
feat: implement opensearch serverless support #212
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #212 +/- ##
==========================================
- Coverage 93.91% 93.82% -0.09%
==========================================
Files 18 19 +1
Lines 5654 5766 +112
Branches 136 152 +16
==========================================
+ Hits 5310 5410 +100
- Misses 344 356 +12 ☔ View full report in Codecov by Sentry. |
|
||
beforeAll(() => { | ||
const ragTestStack = new cdk.Stack(undefined, undefined, { | ||
env: { account: cdk.Aws.ACCOUNT_ID, region: cdk.Aws.REGION }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible for this to be a snapshot instead from a mocked or fictitious account and region?
72b4111
to
63db8b2
Compare
f5b95cb
to
469a305
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, next PR increase the code coverage
469a305
to
dd4c5d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes # 136
This PR adds an existingOpensearchServerlessCollection option to support importing existing OpenSearch Serverless collections.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.