chore(pipeline): cleanup before splits introduction - #7085
Merged
lemeurherve merged 2 commits intoJul 17, 2026
Conversation
lemeurherve
added
chore
lemeurherve
force-pushed
the
7037-03-pipeline-cleanup
branch
from
July 16, 2026 21:21
5e5d4ea to
ab15f92
Compare
lemeurherve
commented
Jul 16, 2026
| if (BRANCH_NAME == 'master' || fullTestMarkerFile || weeklyTestMarkerFile || env.CHANGE_ID && (fullTestLabel || weeklyTestLabel)) { | ||
| def branches = [failFast: false] | ||
| lines.each {line -> | ||
| if (line != 'weekly' && (weeklyTestMarkerFile || env.CHANGE_ID && weeklyTestLabel)) { |
Member
Author
There was a problem hiding this comment.
Other lines than weekly are already filtered out in 'parse prep' stage if there is a weekly-test label or marker.
| def branchName = "${repository}:${line}" | ||
| final String branchName = "${repository}:${line}" | ||
| branches[branchName] = { | ||
| def jdk = line == 'weekly' || line == '2.555.x' ? 21 : 17 |
Member
Author
There was a problem hiding this comment.
As we're keeping only the newest and oldest lines, this case never happened.
| // Mark build as failed on any marker file | ||
| def markerErrors = flags.findAll { flag, sources -> 'marker' in sources }.keySet() | ||
| if (!markerErrors.isEmpty()) { | ||
| error "Remember to `git rm ${markerErrors.join(' ')}` before taking out of draft" |
Member
Author
There was a problem hiding this comment.
Making all remaining markers triggering an error, not only full-test marker.
(follow-up of #2166)
| def foundInBuildNumber = 0 | ||
| def archiveExists = false | ||
| def buildNumber = env.BUILD_NUMBER.toInteger() | ||
| int retrieveArtifactsFromPreviousBuilds(String archiveName, String jobName) { |
Member
Author
There was a problem hiding this comment.
Note for later: integrate this function in the pipeline library at the end.
6 tasks
lemeurherve
added
full-test
lemeurherve
added
weekly-test
lemeurherve
marked this pull request as ready for review
July 17, 2026 00:14
lemeurherve
removed
weekly-test
lemeurherve
force-pushed
the
7037-03-pipeline-cleanup
branch
from
July 17, 2026 00:30
8f84fd0 to
a82b5b5
Compare
lemeurherve
force-pushed
the
7037-03-pipeline-cleanup
branch
from
July 17, 2026 00:31
a82b5b5 to
195b9fc
Compare
6 tasks
Member
Author
|
Tested in various pipelines, merging to continue on #7037 extraction and jenkins-infra/helpdesk#5208 Valuable ones incoming:
I'll address any post-merge suggestion in follow-up PRs if needed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change is a cleanup of the pipeline after #7033, #7077, #7078 & #7079 to get a clean state before introducing #7086.
All variables (except
junitResults) are now typed.Labels and markers are regrouped in a
flagsmap instead of having each of them a boolean.Checkout, initialisation and
infra.maybePublishIncrementals()have now their own stages for easier duration retrieval.Extracted from:
Refs:
Testing done
CI
Stage
initandpublish incrementals:Details
Skipped (shown as "..." / gray in the graph):
Initial build, with
weekly-testandlimited-plugin-setlabels: https://ci.jenkins.io/job/Tools/job/bom/job/PR-7085/3Details
Subsequent build triggered with "Build now" via Jenkins UI, with
full-testandlimited-plugin-setlabels: https://ci.jenkins.io/job/Tools/job/bom/job/PR-7085/12Details
Next build triggered via a rerun, with
weekly-testandlimited-plugin-setlabels: https://ci.jenkins.io/job/Tools/job/bom/job/PR-7085/13Details
Final build, no labels: https://ci.jenkins.io/job/Tools/job/bom/job/PR-7085/18
Details
Submitter checklist