-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Gitlab integration support #237
Conversation
GitLab merge request hook payload doesn't provide any information regarding source commits. And since Drone currently doesn't support manual fetch from another remote upstream, merge request hook only supported for maerge request fro within the same repository.
May parse hook payload only once.
Looks like a really solid implementation! Great job! There was an issue with the build related to the go-gitlab client. Take a look and let me know how to proceed. Thanks! |
This should be built with plouc/go-gitlab-client#8 applied. To build it manually: go get github.com/plouc/go-gitlab-client
cd $GOPATH/src/github.com/plouc/go-gitlab-client
git remote add fudanchii https://github.com/fudanchii/go-gitlab-client
git fetch fudanchii
git checkout -b raw-request fudanchii/raw-request
cd $GOPATH/src/github.com/drone/drone
make
Since this is temporary, |
sorry, I should have read your comments more closely :) vendoring packages merge requests We should add issues for any known issues / glitches so that we can track going forward. Again, great work on this! |
I also forgot to mention this package manager: |
… import path. At least until plouc/go-gitlab-client#8 accepted or Drone officially fork it to `drone/go-gitlab` or when we have package management which can do something like this: gom 'github.com/plouc/go-gitlab-client', :github => 'fudanchii/go-gitlab-client', :branch => 'raw-request'
I pushed some workaround for this. Refer to my last commit message. |
@fudanchii i send small PR, to add support self-signed certificates fudanchii/go-gitlab-client#1 |
@Bugagazavr It's been merged. Thank you! |
Gitlab integration support
Merged! Thanks again and feel free to add issues for any outstanding items |
Thanks for supporting GitLab @fudanchii and @bradrydzewski ! |
@dosire You're welcome! We've just started, so it might still rough at the edges. :D Kudos for all of them! |
thanks @fudanchii ;) |
Great work folks! |
When will this feature be live? (on https://drone.io/) |
This is the preliminary version of GitLab integration
Current features:
Known glitch (wip/open for contribution):
This may be fixed at go-gitlab-client ( Currently there's no function similar to go-github's
CreateUpdate
)/
on its name may be broken,Need to encode
/
to%2F
when calling GitLab APIUnlike GitHub, GitLab doesn't provide MR/PR refs at parent repository,
This should be fetched and merged manually at build which require Drone to also save source repository url from which the MR/PR came from, and this may bump into another problem when the source repository is marked as private. Possible fix is to propose a patch to GitLab to add refs at parent repository for each PR/MR, similar to how GitHub PR works. (I just noticed that we don't have this feature too for bitbucket, so nvm I guess?)
@bradrydzewski
I'd like to propose to vendorize go-gitlab-client for now and push back to upstream when things get stable here, or maybe a fork? What do you think?
Any inputs will be much appreciated,
Thanks!
(edit: s/private_key/private_token/g)