Skip to content

Conversation

@zbynek
Copy link

@zbynek zbynek commented Sep 14, 2025

Partial fix for JENKINS-76091

Closes #114
Closes #113

Unlike #114 this also updates BOM to latest to resolve some upper bound issues. The latest BOM requires Jenkins 2.504.3 for several components, so the baseline is increased here as well.

Testing done

Tested in runtime with latest release of GitLab branch source and confirmed that it fails to scan branches with an unresolved symbol exception. Tested in runtime with an incremental build of GitLab branch source and confirmed that it scans branches successfully. Expect to see runtime failures in gitlab-oauth plugin when the current release of gitlab-oauth plugin is used with this incremental build.

Ran automated tests of gitlab-oauth plugin and gitlab-logo plugin with this build

Automated tests passing.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • [n/a] Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@zbynek zbynek marked this pull request as ready for review September 18, 2025 07:07
@jmini
Copy link
Contributor

jmini commented Sep 21, 2025

@zbynek as the maintainer of GitLab4J I have been asked in issue gitlab4j/gitlab4j-api#1280 to create a 5.8.1 release that is still on the 5.x.x stream and removes the usage of PropertyNamingStrategy to be compatible with newer Jackson version and still be on Jersey 2.

From my understanding some installations are broken today (see jenkinsci/gitlab-branch-source-plugin#504)

Maybe you can use this 5.8.1 version as a emergency fix until you have found a way forward with Jersey 3.

@jonesbusy jonesbusy mentioned this pull request Sep 21, 2025
6 tasks
@jonesbusy
Copy link
Contributor

Thanks! Not a maintainer, so not my decision

In the meanwhile I've open #122

Copy link
Member

@alecharp alecharp left a comment

Choose a reason for hiding this comment

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

This still requires a release of jersey3-api plugin and jenkinsci/jackson2-api-plugin#312 to be merged.

pom.xml Outdated
Comment on lines 141 to 144
<repository>
<id>incrementals</id>
<url>https://repo.jenkins-ci.org/incrementals/</url>
</repository>
Copy link
Member

Choose a reason for hiding this comment

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

That shouldn't be required as there is

-Pconsume-incrementals

@alecharp
Copy link
Member

alecharp commented Oct 6, 2025

@zbynek do you think you can revisit this pull request?

@zbynek
Copy link
Author

zbynek commented Oct 6, 2025

@alecharp thanks for your feedback, I'll look into those suggestions once jersey3-api is released.

@alecharp
Copy link
Member

alecharp commented Oct 8, 2025

@alecharp thanks for your feedback, I'll look into those suggestions once jersey3-api is released.

thanks. I asked to adopt the plugin so I could release it. normally, next week is the end of the 2weeks notice so I will release the plugin at the latest by the 20th.

zbynek and others added 3 commits October 8, 2025 20:08
Co-authored-by: Adrien Lecharpentier <[email protected]>
Co-authored-by: Adrien Lecharpentier <[email protected]>
Co-authored-by: Adrien Lecharpentier <[email protected]>
Copy link
Member

@alecharp alecharp left a comment

Choose a reason for hiding this comment

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

Some tweak about versions available in BOM but looks good to me. Thanks @zbynek for your work on this.

@MarkEWaite MarkEWaite self-requested a review October 16, 2025 13:44
@MarkEWaite MarkEWaite changed the title Update gitlab4j to 6.1.0, update Jenkins BOM, require Jenkins 2.479.3 Update gitlab4j to 6.1.0, update Jenkins BOM, require Jenkins 2.504.3 Oct 23, 2025
@MarkEWaite MarkEWaite added breaking Breaking change enhancement New feature or request labels Oct 24, 2025
@MarkEWaite
Copy link
Contributor

Due to the breaking changes in the transition from GitLab API 5.8.x to GitLab API 6.x, we need to plan a coordinated release of this pull request and the dependent pull requests:

If a user installs the new release of GitLab API plugin that includes GitLab API 6.x, without updating any of the other installed plugins, those installed plugins will likely be broken and report errors.

I will be out of the office from Oct 27 - Nov 7, so won't be able to help coordinate the release of those plugins until after I return. Maintainers of those plugins need to all agree that they will release a new version soon after the release of this pull request.

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Changes look good to me. Release needs to be coordinated with the 3 plugins that depend on this plugin.

MarkEWaite added a commit to MarkEWaite/bom that referenced this pull request Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants