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

Require 2.479.1 or newer #235

Merged
merged 1 commit into from
Feb 28, 2025
Merged

Require 2.479.1 or newer #235

merged 1 commit into from
Feb 28, 2025

Conversation

basil
Copy link
Member

@basil basil commented Feb 25, 2025

No description provided.

@basil basil added the internal label Feb 25, 2025
@basil basil requested a review from a team as a code owner February 25, 2025 20:16
<!-- Last 2.426.x BOM version before a trilead api upgrade -->
<!-- Trilead API upgrade breaks plugin compatibility tests due to poor handling of optional dependencies -->
<!-- Do not change this without checking plugin BOM passes with new version -->
<!-- https://github.com/jenkinsci/bom/pull/3404 -->
Copy link
Member

Choose a reason for hiding this comment

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

jenkinsci/bom#3404 obsolete I hope?

Copy link
Member

Choose a reason for hiding this comment

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

@MarkEWaite you added this comment in #212. I lack any context for the problem.

Copy link
Contributor

@MarkEWaite MarkEWaite Feb 28, 2025

Choose a reason for hiding this comment

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

I think that the BOM contributing guide describes the problem as:

Failed to load - update required

This message usually shows that an optional dependency of a plugin is not being updated by the Plugin Compatibility Tester (PCT) and that update is needed by another plugin.

That section includes more details and links to pull requests that have adapted to the issue.

Copy link
Contributor

Choose a reason for hiding this comment

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

I confirmed that the incremental build 354.v22ca_383c6973 from this branch passes PCT with this command:

LINE=weekly PLUGINS=ssh-credentials TEST=InjectedTest bash local-test.sh

The change I locally applied was:

diff --git a/bom-weekly/pom.xml b/bom-weekly/pom.xml
index 9dc987f6..8c8996cf 100644
--- a/bom-weekly/pom.xml
+++ b/bom-weekly/pom.xml
@@ -1471,7 +1471,7 @@
       <dependency>
         <groupId>org.jenkins-ci.plugins</groupId>
         <artifactId>ssh-credentials</artifactId>
-        <version>349.vb_8b_6b_9709f5b_</version>
+        <version>354.v22ca_383c6973</version>
       </dependency>
       <dependency>
         <groupId>org.jenkins-ci.plugins</groupId>

Removing that old comment looks right to me. Thanks!

@alecharp alecharp merged commit 9be5bcd into jenkinsci:master Feb 28, 2025
17 checks passed
@MarkEWaite
Copy link
Contributor

Pull request needs to be labeled "feature" or "developer" or "rfe" in order to create a new release.

@jglick
Copy link
Member

jglick commented Feb 28, 2025

Labeling it after the fact does not produce a release, unless another PR is merged later.

In general, if a PR of this type needs a release in a timely fashion (for example to resolve some issue with bom), please label it as developer, not internal / chore / dependencies etc. If you forget to do so prior to merge, request a manual release. (There was no such request expressed here so far.)

@MarkEWaite
Copy link
Contributor

request a manual release

Thanks! I'd like a manual release of this plugin so that the Spring Security 6.x cleanup work can continue its forward progress. It is not a critical need, but it will help us continue to make progress.

@jglick
Copy link
Member

jglick commented Feb 28, 2025

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

Successfully merging this pull request may close these issues.

4 participants