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

compare workspace branches #10

Open
wants to merge 42 commits into
base: feature/optional-workspaces-params-in-repository
Choose a base branch
from

Conversation

SuZhou-Joe
Copy link
Owner

Description

Issues Resolved

Screenshot

Testing the changes

Check List

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

…earch-project#5944)

* fix missing param and add changelog and test

Signed-off-by: Lu Yu <[email protected]>

* change pr number

Signed-off-by: Lu Yu <[email protected]>

* remove dangling changelog

Signed-off-by: Lu Yu <[email protected]>

---------

Signed-off-by: Lu Yu <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Feb 27, 2024

Codecov Report

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

Project coverage is 66.97%. Comparing base (790c076) to head (8a98150).
Report is 27 commits behind head on feature/optional-workspaces-params-in-repository.

Files Patch % Lines
...rc/core/server/saved_objects/routes/bulk_create.ts 0.00% 2 Missing ⚠️
src/core/server/saved_objects/routes/create.ts 0.00% 2 Missing ⚠️
...ved_objects/service/lib/search_dsl/query_params.ts 66.66% 1 Missing and 1 partial ⚠️
src/core/server/saved_objects/routes/export.ts 0.00% 1 Missing ⚠️
...ore/server/saved_objects/service/lib/repository.ts 80.00% 0 Missing and 1 partial ⚠️
src/core/server/saved_objects/service/lib/utils.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                                 Coverage Diff                                  @@
##           feature/optional-workspaces-params-in-repository      #10      +/-   ##
====================================================================================
- Coverage                                             67.00%   66.97%   -0.04%     
====================================================================================
  Files                                                  3307     3307              
  Lines                                                 63614    63628      +14     
  Branches                                              10163    10170       +7     
====================================================================================
- Hits                                                  42627    42612      -15     
- Misses                                                18518    18524       +6     
- Partials                                               2469     2492      +23     
Flag Coverage Δ
Linux_1 35.21% <ø> (ø)
Linux_2 ?
Linux_3 43.60% <0.00%> (-0.02%) ⬇️
Linux_4 35.18% <0.00%> (-0.02%) ⬇️
Windows_1 35.24% <ø> (ø)
Windows_2 55.06% <52.63%> (+<0.01%) ⬆️
Windows_3 43.62% <0.00%> (-0.02%) ⬇️
Windows_4 35.18% <0.00%> (-0.02%) ⬇️

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.

ashwin-pc and others added 5 commits February 27, 2024 09:18
…passed for the dataSourceId. (opensearch-project#5882)

* [MD] Using legacy client cofig instead of globle config

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] add test cases to test different hostconfig and datasourceId combinations of scenario

Signed-off-by: Xinrui Bai <[email protected]>

* Update changelog file

Signed-off-by: Xinrui Bai <[email protected]>

* Resolve comments

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] update test cases and resolve comments

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] Update unit test

Signed-off-by: Xinrui Bai <[email protected]>

---------

Signed-off-by: Xinrui Bai <[email protected]>
…configuration (opensearch-project#5916)

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

Signed-off-by: Xinrui Bai <[email protected]>

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

Signed-off-by: Xinrui Bai <[email protected]>

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

Signed-off-by: Xinrui Bai <[email protected]>

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

Signed-off-by: Xinrui Bai <[email protected]>

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

Signed-off-by: Xinrui Bai <[email protected]>

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

Signed-off-by: Xinrui Bai <[email protected]>

* Update yml file to disable config

Signed-off-by: Xinrui Bai <[email protected]>

* Update change.md file

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] Fix broken test cases

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] Add more unit tests

Signed-off-by: Xinrui Bai <[email protected]>

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

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] update description of unit test case

Signed-off-by: Xinrui Bai <[email protected]>

* [UT] update unit test cases to resolve comment

Signed-off-by: Xinrui Bai <[email protected]>

* [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 <[email protected]>

---------

Signed-off-by: Xinrui Bai <[email protected]>
SuZhou-Joe and others added 19 commits February 29, 2024 10:07
* feat: add ruanyl as a maintainer

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add CHANGELOG

Signed-off-by: SuZhou-Joe <[email protected]>

---------

Signed-off-by: SuZhou-Joe <[email protected]>
* add BionIT as a maintainer

Signed-off-by: Flyingliuhub <[email protected]>

* update pr link in the changelog.md

Signed-off-by: Flyingliuhub <[email protected]>

---------

Signed-off-by: Flyingliuhub <[email protected]>
…th type (opensearch-project#5997)

* fix schema for test connection

Signed-off-by: Lu Yu <[email protected]>

* add changelog

Signed-off-by: Lu Yu <[email protected]>

---------

Signed-off-by: Lu Yu <[email protected]>
…t#5855)

* Add application configuration service

Signed-off-by: Tianle Huang <[email protected]>

* update API path name

Signed-off-by: Tianle Huang <[email protected]>

* implement two APIs/interfaces

Signed-off-by: Tianle Huang <[email protected]>

* expose get function for other plugins to use

Signed-off-by: Tianle Huang <[email protected]>

* update interfaces

Signed-off-by: Tianle Huang <[email protected]>

* implement the APIs and interfaces

Signed-off-by: Tianle Huang <[email protected]>

* add license and jsdoc

Signed-off-by: Tianle Huang <[email protected]>

* update docs

Signed-off-by: Tianle Huang <[email protected]>

* add more docs

Signed-off-by: Tianle Huang <[email protected]>

* update variable name

Signed-off-by: Tianle Huang <[email protected]>

* remove unnecessary dependency

Signed-off-by: Tianle Huang <[email protected]>

* format readme

Signed-off-by: Tianle Huang <[email protected]>

* use osd version

Signed-off-by: Tianle Huang <[email protected]>

* remove debugging info

Signed-off-by: Tianle Huang <[email protected]>

* update logging

Signed-off-by: Tianle Huang <[email protected]>

* remove lint js

Signed-off-by: Tianle Huang <[email protected]>

* remove logs

Signed-off-by: Tianle Huang <[email protected]>

* update name style

Signed-off-by: Tianle Huang <[email protected]>

* update

Signed-off-by: Tianle Huang <[email protected]>

* update function visibility and error function

Signed-off-by: Tianle Huang <[email protected]>

* fix unit test failures

Signed-off-by: Tianle Huang <[email protected]>

* add unit test

Signed-off-by: Tianle Huang <[email protected]>

* remove lint file

Signed-off-by: Tianle Huang <[email protected]>

* add more tests

Signed-off-by: Tianle Huang <[email protected]>

* add unit tests for routes

Signed-off-by: Tianle Huang <[email protected]>

* add remaining unit tests

Signed-off-by: Tianle Huang <[email protected]>

* add enabled to this plugin

Signed-off-by: Tianle Huang <[email protected]>

* update readme to mention experimental

Signed-off-by: Tianle Huang <[email protected]>

* update change log

Signed-off-by: Tianle Huang <[email protected]>

* dummy commit to trigger workflow rerun

Signed-off-by: Tianle Huang <[email protected]>

* remove experimental

Signed-off-by: Tianle Huang <[email protected]>

* add key to yml file

Signed-off-by: Tianle Huang <[email protected]>

* remove i18n

Signed-off-by: Tianle Huang <[email protected]>

* remove lint rc

Signed-off-by: Tianle Huang <[email protected]>

* update comment style

Signed-off-by: Tianle Huang <[email protected]>

* add input validation

Signed-off-by: Tianle Huang <[email protected]>

* update unit tests

Signed-off-by: Tianle Huang <[email protected]>

* prevent multiple registration

Signed-off-by: Tianle Huang <[email protected]>

* add return types

Signed-off-by: Tianle Huang <[email protected]>

* update readme wording

Signed-off-by: Tianle Huang <[email protected]>

* add unit test to the plugin class about double register

Signed-off-by: Tianle Huang <[email protected]>

* move related ymls

Signed-off-by: Tianle Huang <[email protected]>

* move validation to a function

Signed-off-by: Tianle Huang <[email protected]>

* use trimmed versions

Signed-off-by: Tianle Huang <[email protected]>

* reword changelog entry

Signed-off-by: Tianle Huang <[email protected]>

* readability

Signed-off-by: Tianle Huang <[email protected]>

* add back yml change

Signed-off-by: Tianle Huang <[email protected]>

---------

Signed-off-by: Tianle Huang <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
@SuZhou-Joe SuZhou-Joe force-pushed the feature/workspace-parameters branch from 6752994 to 02c8859 Compare March 2, 2024 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants