-
Notifications
You must be signed in to change notification settings - Fork 21
[JENKINS-76091] Use gitlab4j 5.8.1 #122
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
Conversation
|
I've replayed the build using virtual machine agents as a way to avoid issue 4807 |
|
I tested with a locally built version. I confirmed that I can see the problem without this change and that the problem is fixed with this change. We can ask the issue submitters to test it further. |
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.
Thanks!
|
I've posted instructions to each of the duplicate issues in hopes that the submitters will help test the change. The issues include: |
Needs permission from one of the existing maintainers: * @baymac * @surenpi * @jetersen * @jeffpearce * @justinharringa Pull request to be merged and released is: * jenkinsci/gitlab-api-plugin#122 Adoption needs to happen quickly because we need a release of the GitLab API plugin to resolve the issue caused by the Jackson 2.20.0 API update of the SNAKE_CASE symbol. Jenkins issue reports include: * [JENKINS-76091](https://issues.jenkins.io/browse/JENKINS-76091) * [JENKINS-76105](https://issues.jenkins.io/browse/JENKINS-76105) * [JENKINS-76110](https://issues.jenkins.io/browse/JENKINS-76110) * [JENKINS-76116](https://issues.jenkins.io/browse/JENKINS-76116) * [JENKINS-76117](https://issues.jenkins.io/browse/JENKINS-76117) * [JENKINS-76119](https://issues.jenkins.io/browse/JENKINS-76119) * [JENKINS-76125](https://issues.jenkins.io/browse/JENKINS-76125)
|
Thank you. Any pointers on how/when this new release is included in the next release of the BOM?: <dependencies>
<dependency>
<groupId>io.jenkins.tools.bom</groupId>
<artifactId>bom-${jenkins.baseline}.x</artifactId>
<version>5054.v620b_5d2b_d5e6</version>
<type>pom</type>
<scope>import</scope>
</dependency>For the |
|
Bom is released weekly. Version can be overriding like https://github.com/jenkinsci/gitlab-branch-source-plugin/pull/508/files |
Is there a specific concern that you have that should push us to release a new version of the BOM earlier than next Friday? Version override is easy to do if someone needs it (as noted by @jonesbusy ) and Jenkins runtime allows newer plugin versions to be installed and used without requiring a change to the plugin BOM. |
|
Working with version override is OK for me as well. Thank you! |
JENKINS-76091 Use gitlab4j 5.8.1 to fix SNAKE_CASE missing symbol
#121 (comment)
To work with Jackson 2.20 without migrating to Jersey 3
Testing done
mvn clean install
Verified failure in GitLab branch source without this change and confirmed that the failure does not happen when this change is used.
Submitter checklist