-
Notifications
You must be signed in to change notification settings - Fork 205
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
Consider a tag's version when cleaning up job results #5146
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5146 +/- ##
=======================================
Coverage 98.27% 98.27%
=======================================
Files 388 388
Lines 36712 36730 +18
=======================================
+ Hits 36080 36098 +18
Misses 632 632
|
0e976b1
to
1b9dfb3
Compare
This pull request is now in conflicts. Could you fix it? 🙏 |
@Mergifyio rebase |
❌ Base branch update has failedGit reported the following error:
err-code: 6049E |
When cleaning up job results, jobs being part of an important build are treated differently. So far really only the BUILD variable is used to decide whether a job belongs to an important build - even if the according tag includes a version number. With this change the version from the tag is considered as well and matched against the job's VERSION variable. When a tag only specifies the BUILD then it is still just matched against the job's BUILD variable. Related ticket/note: https://progress.opensuse.org/issues/129244#note-18
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 did not find any errors but I would like others to review as well
When cleaning up job results, jobs being part of an important build are treated differently. So far really only the BUILD variable is used to decide whether a job belongs to an important build - even if the according tag includes a version number.
With this change the version from the tag is considered as well and matched against the job's VERSION variable.
Related ticket/note: https://progress.opensuse.org/issues/129244#note-18