Datasources Integ Test Fix by vamsimanohar · Pull Request #1541 · opensearch-project/sql · GitHub
Skip to content

Datasources Integ Test Fix#1541

Merged
vamsimanohar merged 1 commit into
opensearch-project:mainfrom
vamsimanohar:ds-integ-test-fix
Apr 18, 2023
Merged

Datasources Integ Test Fix#1541
vamsimanohar merged 1 commit into
opensearch-project:mainfrom
vamsimanohar:ds-integ-test-fix

Conversation

@vamsimanohar

@vamsimanohar vamsimanohar commented Apr 18, 2023

Copy link
Copy Markdown
Member

Description

  • with the current state, integ-state fails when cluster restricts actions on system index .ql-datasources.
  • In order to avoid this, moved populating datasources using APIs instead of loading data directly in .ql-datasources.

Issues Resolved

#1538

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

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.

@vamsimanohar vamsimanohar linked an issue Apr 18, 2023 that may be closed by this pull request
Signed-off-by: vamsi-amazon <reddyvam@amazon.com>
@codecov-commenter

Copy link
Copy Markdown

@vamsimanohar vamsimanohar merged commit 4e9ca01 into opensearch-project:main Apr 18, 2023
opensearch-trigger-bot Bot pushed a commit that referenced this pull request Apr 18, 2023
(cherry picked from commit 4e9ca01)
opensearch-trigger-bot Bot pushed a commit that referenced this pull request Apr 18, 2023
(cherry picked from commit 4e9ca01)
vamsimanohar added a commit that referenced this pull request Apr 18, 2023
(cherry picked from commit 4e9ca01)

Co-authored-by: Vamsi Manohar <reddyvam@amazon.com>
vamsimanohar added a commit that referenced this pull request Apr 18, 2023
(cherry picked from commit 4e9ca01)

Co-authored-by: Vamsi Manohar <reddyvam@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.x backport 2.7 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AUTOCUT] Integration Test failed for sql: 2.7.0 tar distribution

4 participants