-
Notifications
You must be signed in to change notification settings - Fork 528
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
Use SubTask.getOwnerExecutable
#2388
Conversation
…into getOwnerExecutable
the build is failing because of |
jenkinsci#2379, and fixing `github` dep versions
…into getOwnerExecutable
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.
@car-roll FYI
<dependency> | ||
<groupId>org.hamcrest</groupId> | ||
<artifactId>hamcrest</artifactId> | ||
<version>${hamcrest.version}</version> |
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.
see #2444
Checked locally with mvnd clean install dependency:sources -Dtest=InjectedTest -DfailIfNoTests=false so I hope this can finally go green before it gets more merge conflicts! |
I merged it but since this commit ATH test fail in timeout. |
Perhaps from the Jenkins core or BOM updates? https://ci.blueocean.io/job/blueocean/job/master/2181/execution/node/65/log/ shows lots of stuff indicating mistakes in test setup
Anyway should you not turn off ci.blueocean.io by now? https://ci.blueocean.io/job/blueocean/job/master/ has not been stable since November. |
yup there is an issue here jenkins-infra/helpdesk#2954 |
* Use `SubTask.getOwnerExecutable` * Update to releases * Removing some gratuitous version overrides * Using newest available BOM * Restoring `InjectedTest`, deleting Animal Sniffer config obsolete as of jenkinsci#2379, and fixing `github` dep versions * May as well go to 2.401.x now
@@ -32,12 +32,11 @@ | |||
when changing jenkins core version please remember to change it in Jenkinsfile as well jenkinsVersions | |||
and acceptance-tests/runner/scripts/args.sh | |||
--> | |||
<jenkins.version>2.361.4</jenkins.version> |
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.
Implies a corresponding change in jenkinsci/bom/bom-2.387.x/pom.xml
: jenkinsci/bom#2395
Downstream of jenkinsci/jenkins#7599 & jenkinsci/workflow-durable-task-step-plugin#287.