[Backport 2.11] [Feature] Datasource selector of multiple datasources #5236
Conversation
This file contains 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
![opensearch-trigger-bot[bot]](https://avatars.githubusercontent.com/u/80134844?s=60&v=4){kind=small}
* initial commit for datasource Signed-off-by: Eric <[email protected]> * initial commit for datasource service Signed-off-by: Eric <[email protected]> * [Data Sources] Move datasource codebase from datasource plugin to data plugin Signed-off-by: Eric <[email protected]> * [Data Sources] Add datasource factory Signed-off-by: Eric <[email protected]> * datasource service Signed-off-by: Eric <[email protected]> * datasource with factory Signed-off-by: Eric <[email protected]> * datasource selector Signed-off-by: Eric <[email protected]> * exposes datasources from data plugin Signed-off-by: Eric <[email protected]> * index pattern datasource registration Signed-off-by: Eric <[email protected]> * add datasource selector to discover Signed-off-by: Eric <[email protected]> * remove unused files Signed-off-by: Eric <[email protected]> * add/exposes types for datasources as a set of services Signed-off-by: Eric <[email protected]> * add datasource selector unit tests Signed-off-by: Eric <[email protected]> * add metadata to selectable Signed-off-by: Eric <[email protected]> * redirection to observability for non-index-pattern datasource Signed-off-by: Eric <[email protected]> * add datasource factory tests Signed-off-by: Eric <[email protected]> * add datasource service test Signed-off-by: Eric <[email protected]> * datasources related fixes Signed-off-by: Eric <[email protected]> * add tests for datasource selectable Signed-off-by: Eric <[email protected]> * added types/interfaces for sidebar selector, and add a couple of enhancements Signed-off-by: Eric <[email protected]> * remove pill effect Signed-off-by: Eric <[email protected]> * change type for name display in selector for index patterns Signed-off-by: Eric <[email protected]> * remove legacy index selector Signed-off-by: Eric <[email protected]> * rename datasource change handler Signed-off-by: Eric <[email protected]> * remove unused constants Signed-off-by: Eric <[email protected]> * fix a redirection bug Signed-off-by: Eric <[email protected]> * add to change log Signed-off-by: Eric <[email protected]> * fix bg color issue in source selector Signed-off-by: Eric <[email protected]> * address oui and missing guard Signed-off-by: Eric <[email protected]> * add test subj Signed-off-by: Eric <[email protected]> * i18 and datasource interface default return Signed-off-by: Eric <[email protected]> * add default datasource tests Signed-off-by: Eric <[email protected]> * fix typo Signed-off-by: Eric <[email protected]> * modify wording Signed-off-by: Eric <[email protected]> * add experimental annotation to datasource Signed-off-by: Eric <[email protected]> * add type datasource filtering to remove type error hint Signed-off-by: Eric <[email protected]> * remove unused type Signed-off-by: Eric <[email protected]> * add 'type' to option type Signed-off-by: Eric <[email protected]> * Type fixes Signed-off-by: Ashwin P Chandran <[email protected]> * cherry-pick type changes from remote and add partial fixes Signed-off-by: Eric <[email protected]> * add one type exports Signed-off-by: Eric <[email protected]> * remaining type errors fixed Signed-off-by: Ashwin P Chandran <[email protected]> * addressing comments Signed-off-by: Eric <[email protected]> * address dedup Signed-off-by: Eric <[email protected]> * refactor datasource_selectable to address comments Signed-off-by: Joshua Li <[email protected]> * remove unnecessary optional chaining Signed-off-by: Joshua Li <[email protected]> * refactor variable names Signed-off-by: Joshua Li <[email protected]> * move functnions out of selectable component Signed-off-by: Eric <[email protected]> * add comments for dedup/options list updates Signed-off-by: Eric <[email protected]> * add experimental annotation Signed-off-by: Eric <[email protected]> * callback and experimental annotation on types Signed-off-by: Eric <[email protected]> * handleSourceSelection callback Signed-off-by: Eric <[email protected]> * datasource annotation and import adjustment Signed-off-by: Eric <[email protected]> * remove pill effect Signed-off-by: Eric <[email protected]> * minor changes addressing latest comments Signed-off-by: Eric <[email protected]> * remove unused tests Signed-off-by: Eric <[email protected]> --------- Signed-off-by: Eric <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Joshua Li <[email protected]> Co-authored-by: Josh Romero <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Joshua Li <[email protected]> (cherry picked from commit 1e980fa) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot
bot
requested review from
ananzh and
kavilla
as code owners
opensearch-trigger-bot
bot
added
the
autocut
opensearch-trigger-bot
bot
requested a review
from seanneumann
as a code owner
opensearch-trigger-bot
bot
requested review from
ashwin-pc,
joshuarrrr,
abbyhu2000,
zengyan-amazon,
kristenTian,
zhongnansu,
manasvinibs,
ZilongX,
Flyingliuhub and
BSFishy
as code owners
joshuarrrr
approved these changes
Oct 6, 2023
Codecov Report
@@ Coverage Diff @@
## 2.11 #5236 +/- ##
===========================================
+ Coverage 43.76% 66.81% +23.05%
===========================================
Files 1552 3284 +1732
Lines 31286 63146 +31860
Branches 5911 10047 +4136
===========================================
+ Hits 13691 42194 +28503
- Misses 16572 18482 +1910
- Partials 1023 2470 +1447
Flags with carried forward coverage won't be shown. Click here to find out more.
|
seanneumann
approved these changes
Oct 6, 2023
71 tasks
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.
Backport 1e980fa from #5167.