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

Update baseline to 2.414.x #515

Merged
merged 5 commits into from
Mar 4, 2024
Merged

Update baseline to 2.414.x #515

merged 5 commits into from
Mar 4, 2024

Conversation

jglick
Copy link
Member

@jglick jglick commented Feb 29, 2024

@jglick jglick requested a review from a team as a code owner February 29, 2024 20:05
@jglick jglick added the dependencies Pull requests that update a dependency file label Feb 29, 2024
[ platform: "linux", jdk: "11"],
[ platform: "linux", jdk: "17"],
[ platform: "windows", jdk: "11"]
/*
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated from archetypes.

<artifactId>bom-2.361.x</artifactId>
<version>2102.v854b_fec19c92</version>
<artifactId>bom-2.414.x</artifactId>
<version>2839.v003b_4d9d24fd</version>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also currently latest BOM for this line.

@jglick
Copy link
Member Author

jglick commented Mar 1, 2024

Not sure what is causing the addContentsRunningBuild flake. Looks like the sort of thing jenkinsci/workflow-support-plugin#235 was supposed to have fixed. Not reproducible locally (including on Windows 10).

@jglick jglick requested a review from Dohbedoh March 1, 2024 13:43
@@ -91,7 +96,7 @@ public void addContentsRunningBuild() throws Exception {
WorkflowRun workflowRun = Optional.ofNullable(p.scheduleBuild2(0))
.orElseThrow(AssertionFailedError::new)
.waitForStart();
j.waitForMessage("Running on", workflowRun);
SemaphoreStep.waitForStart("wait/1", workflowRun);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cloudbees internal reference: BEE-41313

Comment on lines +197 to +198
SemaphoreStep.success("wait/1", null);
j.waitForCompletion(workflowRun);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BEE-46597

useContainerAgent: true, // Set to `false` if you need to use Docker for containerized tests
configurations: [
[platform: 'linux', jdk: 21],
[platform: 'windows', jdk: 17],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am still baffled why this is the recommended setting. From a cost perspective sure. But I believe there should at least be one combination where just one variable is different.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the place to discuss it; just copying the Jenkinsfile from archetypes.

@jglick jglick added developer and removed dependencies Pull requests that update a dependency file labels Mar 4, 2024
@jglick jglick merged commit f28a13c into jenkinsci:master Mar 4, 2024
16 checks passed
@jglick jglick deleted the baseline branch March 4, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants