Skip to content

Conversation

@alecharp
Copy link
Member

@alecharp alecharp commented Oct 2, 2025

This provides the migration of the GitLab API from version 5 to version 6.
Requires jenkinsci/gitlab-api-plugin#121.

Testing done

Ran mvn verify locally without any problem.

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
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@alecharp alecharp requested a review from jetersen as a code owner October 2, 2025 13:12
@alecharp alecharp marked this pull request as draft October 2, 2025 13:12
@alecharp
Copy link
Member Author

I was able to use the latest build of the latest commit to build https://gitlab.com/MarkEWaite/hugo-experiment without problem.

I used a multibranch pipeline, set GitLab Project source, didn't use any credentials, used MarkEWaite user, it successfully listed all repository. On branch scanning, it discovers 2 MR and a branch, built all of that. It also renamed the project to use GitLab user and repository name.

@MarkEWaite
Copy link
Contributor

MarkEWaite commented Oct 23, 2025

I've tested the incremental build of this pull request with the latest incremental build from:

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 other 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 the API plugin pull request.

@alecharp
Copy link
Member Author

@jetersen we would need to schedule a synchronised release of jenkinsci/gitlab-api-plugin#121 and this pull request. Would you be available perform it (no schedule yet, just trying to prepare it)? If not, maybe I can become a temporary maintainer of the plugin to help you with that? cc @MarkEWaite

@MarkEWaite
Copy link
Contributor

@jetersen we would need to schedule a synchronised release of jenkinsci/gitlab-api-plugin#121 and this pull request. Would you be available perform it (no schedule yet, just trying to prepare it)? If not, maybe I can become a temporary maintainer of the plugin to help you with that? cc @MarkEWaite

I'd like to do the release either this week or next week, but a coordinated release of all the components is much more important than the timing of the release.

@jetersen
Copy link
Member

@alecharp I am available

@alecharp
Copy link
Member Author

alecharp commented Nov 14, 2025

The version of GitLab API plugin was bump to 6.2.0-111.vf174d6b_8c12e which is a release containing jenkinsci/gitlab-api-plugin#121.

Once this was validated by CI, it should be merged and release as soon as possible.

@MarkEWaite
Copy link
Contributor

@alecharp I believe that the bom-2.504.x version needs to be updated to either 5681.v79d2ddf61465 or 5659.vecf9e2dc5a_ed. The CI job is failing with an upper bounds dependency message

@alecharp
Copy link
Member Author

Thanks for catching that @MarkEWaite. We are good now. @jetersen if you can have a look that would be fantastic.

@jetersen jetersen added the bug Something isn't working label Nov 14, 2025
@jetersen jetersen merged commit 1a1e87b into jenkinsci:master Nov 14, 2025
18 checks passed
@alecharp alecharp deleted the chore/migrate-gitlab-api-from-5-to-6 branch November 14, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants