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

Add publishers to update the commit/merge request status in GitLab #192

Conversation

coder-hugo
Copy link
Contributor

This PR moves the configuration (and execution) of the GitLab commit status updates to a publisher ("Publish build status to GitLab commit (GitLab 8.1+ required)"). With this change its possible to trigger builds manually without adding any special parameter and the status of the corresponding commit in GitLab gets updated. Additionally there is a XStream converter that migrates the configuration of old jobs to use this publisher automatically depending on their GitLabPushTrigger's configuration.
There is just one thing missing of the previous functionality: The commit status "pending" will not be sent to GitLab. The reason for this is that I haven't found any extension point that throws an event or something similar if a build was triggered but not yet started.
Fixes #162

@jenkinsadmin
Copy link
Member

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch 3 times, most recently from f19a4f1 to 07245d7 Compare February 13, 2016 16:10
@coder-hugo coder-hugo changed the title Add publishers to update the commit/merge request status in GitLab [WIP] Add publishers to update the commit/merge request status in GitLab Feb 16, 2016
@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch from 07245d7 to 9c62979 Compare February 16, 2016 21:55
@coder-hugo coder-hugo changed the title [WIP] Add publishers to update the commit/merge request status in GitLab Add publishers to update the commit/merge request status in GitLab Feb 16, 2016
@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch 2 times, most recently from fc2d23c to dc018f4 Compare February 18, 2016 22:08
@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch 2 times, most recently from f0c0019 to d9961be Compare February 20, 2016 15:56
@coder-hugo coder-hugo mentioned this pull request Mar 1, 2016
@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch from d9961be to e8768c7 Compare March 2, 2016 20:15
@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch from e8768c7 to fc3ad7b Compare March 10, 2016 20:42
@coder-hugo coder-hugo force-pushed the feature/add-publishers-to-update-gitlab-status branch from fc3ad7b to c457e13 Compare March 10, 2016 21:09
@coder-hugo
Copy link
Contributor Author

Close this as there is no possibility to change the target branch. New PR #229

@coder-hugo coder-hugo closed this Mar 10, 2016
exceed-alae pushed a commit to exceed-alae/gitlab-plugin that referenced this pull request May 20, 2022
exceed-alae pushed a commit to exceed-alae/gitlab-plugin that referenced this pull request May 20, 2022
GhprbPullRequestMerge must NOT persist and serialize logger jenkinsci#192
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants