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

[META] Plugins Integration Testing for Test Automation Surge #476

Closed
owaiskazi19 opened this issue Sep 14, 2021 · 1 comment
Closed

[META] Plugins Integration Testing for Test Automation Surge #476

owaiskazi19 opened this issue Sep 14, 2021 · 1 comment
Assignees
Labels
enhancement New Enhancement

Comments

@owaiskazi19
Copy link
Member

owaiskazi19 commented Sep 14, 2021

Run the Integration Test cases in the OpenSearch distribution building workflow for the plugins

Successful:

Plugin Name Result
AD Plugin
  • With security
  • without security
SQL Plugin
  • With security
  • without security
KNN Plugin
  • With security
  • without security
Async-search
  • With security
  • without security

Changes required:

Plugin Name Result Issue Fix
Alerting Plugin
  • With security
  • without security
Since we are downloading alerting plugin as a common dependency for other plugins, the logic to run integ test needs to be changed to run alerting as an individual plugin Fixed in #512
Notebooks
  • With security
  • without security
Not able to find /gradlew to run integ test as the plugin resides in https://github.com/opensearch-project/dashboards-notebooks/tree/main/opensearch-notebooks Fixed in #529
Reports Scheduler
  • With security
  • without security
Not able to find /gradlew to run integ test as the plugin resides in https://github.com/opensearch-project/dashboards-reports/tree/main/reports-scheduler Fixed in #529

Reference for Plugins: #58

@owaiskazi19 owaiskazi19 added the enhancement New Enhancement label Sep 14, 2021
@minalsha
Copy link

minalsha commented Sep 14, 2021

@owaiskazi19 can you also document the error you are facing and what changes are required for successfully running Alerting/notebooks and Reports Scheduler plugins. Also, please document the resolution shared by @dblock to address this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New Enhancement
Projects
None yet
Development

No branches or pull requests

3 participants