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

Add more details to integTest failure issues #545

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

gaiksaya
Copy link
Member

@gaiksaya gaiksaya commented Oct 25, 2024

Description

This change adds below details to the existing integration issue logic:

  • Add build number
  • Add RC number
  • Add metrics visualization URLs per version and component to get exact failures
  • Tag release owners if one is assigned to the release issue

image

Issues Resolve

opensearch-project/opensearch-build#5122

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.

Copy link

codecov bot commented Oct 25, 2024

Codecov Report

Attention: Patch coverage is 86.84211% with 5 lines in your changes missing coverage. Please review.

Project coverage is 84.72%. Comparing base (b7e7b62) to head (6d44ab4).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
tests/jenkins/TestReleaseMetricsData.groovy 71.42% 1 Missing and 3 partials ⚠️
src/jenkins/ReleaseMetricsData.groovy 93.75% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main     #545   +/-   ##
=========================================
  Coverage     84.72%   84.72%           
- Complexity      105      114    +9     
=========================================
  Files           115      117    +2     
  Lines           648      681   +33     
  Branches         71       75    +4     
=========================================
+ Hits            549      577   +28     
- Misses           29       30    +1     
- Partials         70       74    +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Sayali Gaikawad <[email protected]>
@gaiksaya gaiksaya merged commit c63a248 into opensearch-project:main Oct 25, 2024
9 checks passed
@gaiksaya gaiksaya deleted the integTest branch October 25, 2024 20:15
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 25, 2024
Signed-off-by: Sayali Gaikawad <[email protected]>
(cherry picked from commit c63a248)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants