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

[MD]Re-import @opensearch-project/opensearch@^2.1.0 from alias #3471

Conversation

zhongnansu
Copy link
Member

@zhongnansu zhongnansu commented Feb 21, 2023

Inspired by #3469

  • Alias @opensearch-project/opensearch@^2.1.0 to @openearch-project/opensearch-next
  • Re-import modules in Multiple DataSource plugin to re-import from the aliases

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

…earch-project/opensearch-next

Signed-off-by: Su <szhongna@amazon.com>
@codecov-commenter
Copy link

Codecov Report

Merging #3471 (2b5c565) into main (f5ee06d) will not change coverage.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main    #3471   +/-   ##
=======================================
  Coverage   66.43%   66.43%           
=======================================
  Files        3205     3205           
  Lines       61559    61559           
  Branches     9494     9494           
=======================================
  Hits        40896    40896           
  Misses      18394    18394           
  Partials     2269     2269           
Flag Coverage Δ
Linux 66.37% <100.00%> (ø)
Windows 66.38% <100.00%> (ø)

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

Impacted Files Coverage Δ
...plugins/data_source/server/client/client_config.ts 100.00% <ø> (ø)
...c/plugins/data_source/server/client/client_pool.ts 46.66% <ø> (ø)
...gins/data_source/server/client/configure_client.ts 46.66% <ø> (ø)
...ata_source/server/client/configure_client_utils.ts 64.70% <ø> (ø)
...ta_source/server/legacy/configure_legacy_client.ts 59.61% <ø> (ø)
src/plugins/data_source/server/lib/error.ts 100.00% <ø> (ø)
...ource/server/client/configure_client.test.mocks.ts 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@zhongnansu zhongnansu marked this pull request as ready for review February 21, 2023 07:16
@zhongnansu zhongnansu requested a review from a team as a code owner February 21, 2023 07:16
@zhongnansu zhongnansu added multiple datasource multiple datasource project backport 2.x and removed backport 2.x multiple datasource multiple datasource project labels Feb 21, 2023
@zhongnansu zhongnansu marked this pull request as draft February 21, 2023 17:41
@zhongnansu zhongnansu closed this Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants