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

[FEATURE] Metrics to track the integration test failures at the tests level #68

Open
prudhvigodithi opened this issue Aug 27, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@prudhvigodithi
Copy link
Collaborator

Is your feature request related to a problem?

Coming from Central Release Dashboard, track the integration test failures at the tests level (displaying the exact failed tests at component level) and give a report of the failed/flaky tests.

Today we use the libraries publishDistributionBuildResults and publishIntegTestResults to index the build and integration test failures for both OpenSearch and OpenSearch Dashboards. Extend this to ensure the failed integration tests has the information displaying the exact failed tests. Once the failed tests are indexed create visualizations to surface the failures at RC level and find the flaky test trends. Having the failed tests would help the release manager and plugin teams to quickly identify the exact failures of the integration test.

What solution would you like?

For dashboards the cypress reports are stored in cypress/results/ and for OpenSearch plugins in build/reports/tests/integTest/.

  • Create a library to parse these reports and index to the cluster periodically for every component during the integration test execution.
    (or)
  • Use the test report workflow to add the failing tests and classes information to test manifest and update the library publishIntegTestResults to parse the yaml file and index the results.

Do you have any additional context?

Add any other context or screenshots about the feature request here.

@prudhvigodithi prudhvigodithi added enhancement New feature or request untriaged Issues that have not yet been triaged labels Aug 27, 2024
@peterzhuamazon
Copy link
Member

Hi,

I do prefer to keep adding the information to test-report.yml as it could be the single source of truth, instead of directly reading from the cypress/results dir. In the end if we upgrade cypress version (which is likely based on how old cypress9 is), or change to a different test framework later on, we do not need to change the metrics code that much.

Thanks.

@prudhvigodithi
Copy link
Collaborator Author

prudhvigodithi commented Aug 27, 2024

[Triage]
Thanks @peterzhuamazon, yes today we upload all the the gradle and cypress test reports to the build s3 bucket, example

integ-test/2.16.0/10154/linux/x64/tar/test-results/8503/integ-test/k-NN/with-security/opensearch-integ-test/

Screenshot 2024-08-27 at 11 45 46 AM

integ-test-opensearch-dashboards/2.16.0/7856/linux/x64/tar/test-results/6186/integ-test/OpenSearch-Dashboards-ci-group-9/with-security/cypress-report/
Screenshot 2024-08-27 at 11 47 10 AM

Extend the report workflow to collect the above information and update the test-report.yml with all the details.

./report.sh manifests/1.3.19/opensearch-1.3.19-test.yml --artifact-paths opensearch=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/1.3.19/10212/linux/arm64/rpm --test-run-id 8574 --test-type integ-test --base-path https://ci.opensearch.org/ci/dbc/integ-test/1.3.19/10212/linux/arm64/rpm --release-candidate 3 https://ci.opensearch.org/ci/dbc/integ-test/1.3.19/10212/linux/arm64/rpm/test-results/8574/integ-test/k-NN/without-security/k-NN.yml

@peterzhuamazon
Copy link
Member

Just have a offline conversation with @prudhvigodithi , and we both agrees that processing the opensearch-integ-test and cypress results dir for testclass and testname is beneficial to catching the failure, similar to what we did with gradle check failure index.

I will add this as part of the test report manifest soon.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🏗 In progress
Status: Action items ✍
Development

No branches or pull requests

2 participants