Skip to content

Add gitlab integration #420

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

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Add gitlab integration #420

wants to merge 8 commits into from

Conversation

wlcx
Copy link

@wlcx wlcx commented Apr 16, 2025

This is a draft currently as there's some tyre kicking still to do, but I thought it was worth putting up for some initial thoughts!

resolves #329

Still to do:

  • avatars
  • why don't builds get triggered on commit?
  • why aren't statuses working?
    • Nicer status names?
  • probably other stuff
  • squash/rebase as needed

@Mic92
Copy link
Member

Mic92 commented Apr 16, 2025

Looks ok so far.



class CreateGitlabProjectHooks(ThreadDeferredBuildStep):
name = "create_gitlab_project_hooks"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw. if gitlab has a different way to install webhooks i.e. through some app interface, this might be preferred. It's a bit of a historical accident in github that we didn't just use the webhooks that are provided by a github app rather than installing an explicit webhook into each repository.

@greg-hellings
Copy link

Ooo, I'm eager to have this. I've been building all my Nix targets using a dynamic fan-out Pipeline in Gitlab CI right now. It's working OK, but I know there are significant efficiencies that this Buildbot config uses and I can't wait to taste them.

@Mic92
Copy link
Member

Mic92 commented Apr 18, 2025

@greg-hellings maybe you can team up with @wlcx
We have a matrix channel at #buildbot-nix:thalheim.io

@greg-hellings
Copy link

@Mic92 Happily! I can definitely look at the list of things he has above still TBD. I'm in the midst of migrating hypervisors, so you won't see me back on Matrix over this weekend. I should be there Monday to collaborate on this.

@wlcx wlcx force-pushed the main branch 3 times, most recently from b562dce to 017c0a9 Compare April 23, 2025 13:05
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.

Gitlab integration?
3 participants