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 opensearch-2.9.0-test.yml with sql dummy master key for testing. #3738

Merged
merged 1 commit into from
Jul 14, 2023

Conversation

vamsi-amazon
Copy link
Member

@vamsi-amazon vamsi-amazon commented Jul 14, 2023

Description

Added new datasources master key config required for datasource APIs.

Issues Resolved

Fix for Integ test failure opensearch-project/sql#1866 (comment)

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.

Added new datasources master key config required for datasource APIs.

Signed-off-by: Vamsi Manohar <[email protected]>
@codecov
Copy link

codecov bot commented Jul 14, 2023

Codecov Report

Merging #3738 (c39642a) into main (18f221f) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3738      +/-   ##
==========================================
- Coverage   91.54%   91.53%   -0.02%     
==========================================
  Files         182      182              
  Lines        5420     5420              
==========================================
- Hits         4962     4961       -1     
- Misses        458      459       +1     

see 1 file with indirect coverage changes

Copy link
Member

@peterzhuamazon peterzhuamazon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @vamsi-amazon is it ok to show this masterkey? Or is this just a dummy?
Thanks.

@vamsi-amazon
Copy link
Member Author

vamsi-amazon commented Jul 14, 2023

@peterzhuamazon This is dummy one for integ test cluster domain. For customer domains, user has to set this up on their own. Since integ test domains are temporary ones, it won't be an issue.

@peterzhuamazon peterzhuamazon merged commit 82dc403 into opensearch-project:main Jul 14, 2023
60 checks passed
@peterzhuamazon peterzhuamazon changed the title Update opensearch-2.9.0-test.yml Update opensearch-2.9.0-test.yml with sql dummy master key for testing. Jul 14, 2023
@Rishikesh1159 Rishikesh1159 mentioned this pull request Jul 18, 2023
42 tasks
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.

2 participants