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

[JEP-230] [JENKINS-55582] Test instance-identity as a detached plugin #1159

Closed
wants to merge 3 commits into from

Conversation

jglick
Copy link
Member

@jglick jglick commented May 23, 2022

Checking how much mayhem jenkinsci/jenkins#6585 wreaks on PCT. Could either solve with a new PCT hook, or just by updating a handful of plugins after the fact to depend on the new baseline. As per #681 we cannot update the BOM until the oldest supported line has the split.

@jglick
Copy link
Member Author

jglick commented May 23, 2022

Oops, forgot to turn off failFast.

@jglick
Copy link
Member Author

jglick commented May 23, 2022

Some problems with PCT relating to the detached sshd bump in core.

java.io.IOException: Failed to load: Pipeline: API (workflow-api 1143.v2d42f1e9dea_5)
 - Failed to load: Pipeline: Step API (workflow-step-api 622.vb_8e7c15b_c95a_)
 - Update required: SSH server (sshd 3.228.v4c9f9e652c86) to be updated to 3.236.ved5e1b_cb_50b_2 or higher

Not following what is happening here, since #1150 and #1151 already updated this in the BOM.

@jglick
Copy link
Member Author

jglick commented May 23, 2022

Probably jenkinsci/jenkins@d96d64b is interacting poorly with PCT?

@jglick
Copy link
Member Author

jglick commented May 23, 2022

In the case of pipeline-graph-analysis, mvn dependency:tree shows that the old version of sshd is in the test classpath:

org.jenkins-ci.plugins:pipeline-graph-analysis:hpi:999999-SNAPSHOT
+- …
+- org.jenkinsci.plugins:pipeline-model-definition:jar:1.9.3:test
|  +- …
|  +- org.jenkins-ci.plugins.workflow:workflow-cps-global-lib:jar:570.v21311f4951f8:test
|  |  +- …
|  |  +- org.jenkins-ci.plugins:git-server:jar:1.10:test
|  |  |  \- org.jenkins-ci.modules:sshd:jar:3.228.v4c9f9e652c86:test
|  |  |     +- …
|  |  \- …
|  +- …
+- …

@jglick
Copy link
Member Author

jglick commented May 23, 2022

(Possibly a case that #1059 would handle better.)

@jglick jglick mentioned this pull request May 24, 2022
@jglick
Copy link
Member Author

jglick commented Jun 27, 2022

No longer useful I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant