-
Notifications
You must be signed in to change notification settings - Fork 275
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
[IntegTest] Onboard all plugins into automated integration tests for the distribution #58
Comments
I Think in a glance these are missing: Perftop/ Job Schedule / Performance Analyzer/Common Utils and Trace Analytics |
kNN with security test cases have been added for both x64 and arm, please update. |
Updated. Thanks. |
So @bowenlan-amzn has informed me that IM opensearch has Security Enabled since this PR: He also mentions that IM Dashboards tests are now available, but flaky:
|
I have tested Alerting dashboards on OS arm64 manually. |
@lezzago Please make sure to update your automated testing to cover the above (manual testing) scenarios as well. |
What’s the criteria of testing with security vs. without or both? |
Seems like Async Search WITH SEC is up. Need confirmation. |
Seems like SQL Workbench WITH SEC is up. Need confirmation. |
As of 2021/11/22, SQL Workbench has SEC, Notebook dashboards 1.1 and before has SEC, Reports dashboards has SEC, Gantt Charts has NOSEC + SEC. So all the plugins in dashboards now have NOSEC + SEC tests. As of now, everything can use 1.2 branch for 1.2.0 tests, only observability is using main for the test cases (backport later). Dashboards notebooks, trace analytics and observability all merged into trace analytics plugin and repo, will rename to observability after 1.2.0 release. |
@bbarani is there a campaign associated with driving this work? Checking in here for security as we realized this is a gap we'd like to address. Created issue in our repo for tracking opensearch-project/security#1761 CC @davidlago |
Security plugin will provide manual sign-off: opensearch-project/security#1499 (comment). |
Added notificationsDashboards now. |
Please note that this issue is slated to be resolved in 2.1 as this is an enhancement to the current integration tests that have already onboarded for the notification plugin. It might be better to remove this notification issue from this github issue as this current issue revolves around onboarding automated integration tests for the distribution, which Notification plugin already has. The notification github issue mentioned here is just a further enhancement of the existing integration tests. |
Remaining items:
Run the Integration Test cases in the OpenSearch distribution building workflow for all of the plugins
Please make sure your plugin is a part of this manifest file inorder to run integration test via build workflow
⏳ Total Status:
🔎 OpenSearch Plugins:
Notebooks(removed plugin)- [ ] With security- [x] without security- [ ] With security- [x] without security- [x]Job Scheduler(No need to test as other plugin use it already testing it)- [ ] With security- [ ] without security- [ ] With security- [ ] without security- [ ]📊 OpenSearch Dashboards Plugins:
Notebooks Dashboards Plugin(removed plugin combined with trace analytics since 1.2.0 later as observability)- [x] With security- [x] without security- [x] With security- [x] without security- [x]The text was updated successfully, but these errors were encountered: