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

Update sql dummy master key for testing #4007

Merged
merged 2 commits into from
Sep 12, 2023

Conversation

penghuo
Copy link
Contributor

@penghuo penghuo commented Sep 12, 2023

Description

Added new datasources master key config required for datasource APIs.

Issues Resolved

opensearch-project/sql#2082

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Merging #4007 (deef1e4) into main (84c3ba1) will not change coverage.
Report is 3 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4007   +/-   ##
=======================================
  Coverage   92.06%   92.06%           
=======================================
  Files         187      187           
  Lines        5673     5673           
=======================================
  Hits         5223     5223           
  Misses        450      450           

Signed-off-by: Peng Huo <[email protected]>
@gaiksaya
Copy link
Member

gaiksaya commented Sep 12, 2023

Hi @penghuo Can you also update other test manifest? https://github.com/opensearch-project/opensearch-build/blob/main/manifests/2.10.0/opensearch-2.10.0-test.yml

Please ignore looks like it is already present there

Thanks!

@gaiksaya gaiksaya merged commit dba0d1f into opensearch-project:main Sep 12, 2023
42 checks passed
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.

3 participants