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.x] Hide/Show authentication method in multi data source plugin based on configuration #5976

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 20b6c5a from #5916.

…configuration (#5916)

* [AuthType Config] Add EnabledAuthType configuration in yml file and pass value to dataSourceManagement root mount

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [AuthType Config] provide default auth types when types array is empty

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [AuthRegistry Onboard - DataSourceCreationForm] Integrate auth cofig with AuthRegistray and providing data source options from AuthRegistry

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [AuthRegistry Onboard] Support default selected auth type in datasource creation page

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [AuthRegistry Onboard] SDisable auth option selector when auth options no more than 1

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [AuthRegistry Onboard] clear credential attribute every time use select NoAuth option. Also update credentialField data type

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* Update yml file to disable config

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* Update change.md file

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [UT] Fix broken test cases

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [UT] Add more unit tests

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [UT] update unit test and handle scenario when no options enabled

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [UT] update description of unit test case

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [UT] update unit test cases to resolve comment

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

* [UT] Snapshot auth option super selector for unit test. Also upate the default message in datasorurce creation form ---> auth type section

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>

---------

Signed-off-by: Xinrui Bai <xinruiba@amazon.com>
(cherry picked from commit 20b6c5a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
Copy link

codecov bot commented Feb 28, 2024

Codecov Report

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

Project coverage is 67.08%. Comparing base (c659994) to head (00fb52d).

Files Patch % Lines
...rc/plugins/data_source_management/public/plugin.ts 42.85% 1 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              2.x    #5976      +/-   ##
==========================================
+ Coverage   67.06%   67.08%   +0.02%     
==========================================
  Files        3308     3308              
  Lines       63690    63727      +37     
  Branches    10160    10168       +8     
==========================================
+ Hits        42714    42753      +39     
+ Misses      18511    18507       -4     
- Partials     2465     2467       +2     
Flag Coverage Δ
Linux_1 35.21% <ø> (ø)
Linux_2 55.13% <ø> (ø)
Linux_3 44.47% <89.74%> (+0.05%) ⬆️
Linux_4 35.33% <52.17%> (+<0.01%) ⬆️
Windows_1 35.24% <ø> (ø)
Windows_2 55.10% <ø> (ø)
Windows_3 44.49% <89.74%> (+0.07%) ⬆️
Windows_4 35.33% <52.17%> (+<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.

@ZilongX ZilongX merged commit b2d6502 into 2.x Feb 28, 2024
89 of 90 checks passed
@ZilongX ZilongX deleted the backport/backport-5916-to-2.x branch February 28, 2024 18:11
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 multiple datasource multiple datasource project repeat-contributor v2.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants