-
Notifications
You must be signed in to change notification settings - Fork 21
Update gitlab4j to 6.1.0, update Jenkins BOM, require Jenkins 2.504.3 #121
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
base: master
Are you sure you want to change the base?
Conversation
|
@basil I think this is now ready, except for the incremental repo (which should be reverted with |
|
@zbynek as the maintainer of GitLab4J I have been asked in issue gitlab4j/gitlab4j-api#1280 to create a From my understanding some installations are broken today (see jenkinsci/gitlab-branch-source-plugin#504) Maybe you can use this |
|
Thanks! Not a maintainer, so not my decision In the meanwhile I've open #122 |
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.
This still requires a release of jersey3-api plugin and jenkinsci/jackson2-api-plugin#312 to be merged.
pom.xml
Outdated
| <repository> | ||
| <id>incrementals</id> | ||
| <url>https://repo.jenkins-ci.org/incrementals/</url> | ||
| </repository> |
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.
That shouldn't be required as there is
gitlab-api-plugin/.mvn/maven.config
Line 1 in 322a9d1
| -Pconsume-incrementals |
|
@zbynek do you think you can revisit this pull request? |
|
@alecharp thanks for your feedback, I'll look into those suggestions once |
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. |
Co-authored-by: Adrien Lecharpentier <[email protected]>
Co-authored-by: Adrien Lecharpentier <[email protected]>
Co-authored-by: Adrien Lecharpentier <[email protected]>
Co-authored-by: Adrien Lecharpentier <[email protected]>
Removed the incrementals repository from the pom.xml file.
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.
Some tweak about versions available in BOM but looks good to me. Thanks @zbynek for your work on this.
|
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. |
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.
Changes look good to me. Release needs to be coordinated with the 3 plugins that depend on this plugin.
Combine multiple plugin pull requests for compatibility testing. Pull requests include: * jenkinsci/gitlab-api-plugin#121 * jenkinsci/gitlab-branch-source-plugin#511 * jenkinsci/gitlab-logo-plugin#131 * jenkinsci/gitlab-oauth-plugin#201
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