-
Notifications
You must be signed in to change notification settings - Fork 278
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
Add links to test results in the OSD integ test notification #1728
Comments
Currently the url has a random hash which makes it difficult to share. We can pass a test run ID to https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/opensearch-dashboards/integ-test.jenkinsfile#L68 Then the location of test file will use the input instead of random value https://github.com/opensearch-project/opensearch-build/blob/main/src/test_workflow/test_recorder/test_recorder.py#L21 Then we will be able to construct the test result url. |
cc @kavilla |
I think we can just pass cc @dblock |
@tianleh Can you update the current status? |
The issue has a pre-request PR https://github.com/opensearch-project/opensearch-build/pull/2028/files which was waiting for post 2.0 release. I will bring the related PR to sync up with latest main branch. After that, the issue just needs to include the test result link in the message notification. |
@tianleh Can you update the current status? |
Thanks to @zelinh we recently introduced reporting workflow #1274 that generated an overview of all the tests and includes sub-reports that contains links to all test files including videos, stdout, stderr, etc. Closing this issue. Thanks! |
Is your feature request related to a problem? Please describe
Currently only the Jenkins job link is posted in the OSD integ test notification. This is not convenient to look at where the failures are. Since the test result is uploaded under CloudFront, we can include the ci endpoint for test results.
E.g
https://ci.opensearch.org/ci/dbc/integ-test-opensearch-dashboards/1.3.0/1484/linux/x64/test-results/b94f426463b24046a2ed33c5be290508/integ-test/functionalTestDashboards/without-security/test-results/stdout.txt
At minimum, we shall include the url to stdout.txt for
without-security
andwith-security
. We can also consider include links to the video and screenshots.Describe the solution you'd like
At minimum, we shall include the url to stdout.txt for
without-security
andwith-security
. We can also consider include links to the video and screenshots.Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: