-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat: add opensearch provider #5458
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
base: main
Are you sure you want to change the base?
Conversation
|
@n0ta26 is attempting to deploy a commit to the KeepHQ Team on Vercel. A member of the Team first needs to authorize it. |
|
I’ve updated the PR title. |
|
Skipped: This PR does not target one of your configured branches: ( |
shahargl
left a comment
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
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## main #5458 +/- ##
===========================================
- Coverage 46.39% 30.54% -15.86%
===========================================
Files 176 101 -75
Lines 18412 11669 -6743
===========================================
- Hits 8543 3564 -4979
+ Misses 9869 8105 -1764 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@shahargl Thank you so much for reviewing this PR! I noticed there are a few alerts in GitHub Actions, but as far as I can tell they don’t seem to be caused by the changes in this PR. Could you please let me know what the next steps should be on my side? |
Closes #740
📑 Description
This draft PR introduces an OSS OpenSearch provider to Keep and validates it with a basic workflow against a local OpenSearch container.
Changes
✅ Checks
ℹ Additional Information
If the implementation looks good direction-wise, I would like to follow up with documentation updates.
Memo