Skip to content
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

[Backport 2.14] [Backport 2.x] [Discover] Data selector enhancement #6692

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 4d14b60 from #6691.

* datasource and service refactoring

* datasource enhancement and refactoring

* datasource selectable consolidation and refactoring

* add in memory cache with refresh

* move refresh to right side

* renaming

* update default datasource tests

* added more tests for default datasource

* update selector tests

* update changelog

* fix data source service tests

* add and update tests for datasource service

* add more data source service tests

* fix sidebar tests

* add to change log yml

* address comments along with more tests

* add test subject

* reference from correct type path

* correct text

* minor change - remove yet used displayOrder

* remove from changelog as having fragments already

* use expanded name

* fix one test

---------

(cherry picked from commit 42166d0)

Signed-off-by: Eric <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 4d14b60)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Apr 30, 2024

Codecov Report

Attention: Patch coverage is 72.63158% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 67.74%. Comparing base (950f9d8) to head (ed10ca1).

Files Patch % Lines
..._sources/datasource_services/datasource_service.ts 59.25% 9 Missing and 2 partials ⚠️
...rces/datasource_selector/datasource_selectable.tsx 78.78% 3 Missing and 4 partials ⚠️
src/plugins/data/public/plugin.ts 0.00% 4 Missing ⚠️
...lugins/discover/public/__mock__/index.test.mock.ts 25.00% 3 Missing ⚠️
.../data_explorer/public/components/sidebar/index.tsx 75.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             2.14    #6692   +/-   ##
=======================================
  Coverage   67.74%   67.74%           
=======================================
  Files        3412     3414    +2     
  Lines       66879    66915   +36     
  Branches    10864    10868    +4     
=======================================
+ Hits        45306    45331   +25     
- Misses      18935    18946   +11     
  Partials     2638     2638           
Flag Coverage Δ
Linux_1 33.19% <20.68%> (-0.01%) ⬇️
Linux_2 55.66% <ø> (ø)
Linux_4 35.00% <20.68%> (-0.01%) ⬇️
Windows_1 33.22% <20.68%> (-0.01%) ⬇️
Windows_2 55.63% <ø> (ø)
Windows_3 45.28% <72.63%> (+0.02%) ⬆️
Windows_4 35.00% <20.68%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kavilla
Copy link
Member

kavilla commented Apr 30, 2024

@ananzh this got created after the KUI merge and before: #6693 got merged.

The current failure is related to that, do you think that is okay to merge now or should we update this branch and re-run this CI?

@ananzh
Copy link
Member

ananzh commented Apr 30, 2024

let's merge it.

@ananzh ananzh merged commit 1dc54ec into 2.14 Apr 30, 2024
88 of 91 checks passed
@github-actions github-actions bot deleted the backport/backport-6691-to-2.14 branch April 30, 2024 23:57
BionIT pushed a commit to BionIT/OpenSearch-Dashboards that referenced this pull request May 3, 2024
…earch-project#6691) (opensearch-project#6692)

* datasource and service refactoring

* datasource enhancement and refactoring

* datasource selectable consolidation and refactoring

* add in memory cache with refresh

* move refresh to right side

* renaming

* update default datasource tests

* added more tests for default datasource

* update selector tests

* update changelog

* fix data source service tests

* add and update tests for datasource service

* add more data source service tests

* fix sidebar tests

* add to change log yml

* address comments along with more tests

* add test subject

* reference from correct type path

* correct text

* minor change - remove yet used displayOrder

* remove from changelog as having fragments already

* use expanded name

* fix one test

---------

(cherry picked from commit 42166d0)





(cherry picked from commit 4d14b60)

Signed-off-by: Eric <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports data explorer Issues related to the Data Explorer project repeat-contributor v2.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants