Support GitLab for PR Builder

[saadmk11]

Details

As we have initially added support for GitHub, we need to start thinking about supporting other providers such as GitLab.

Initial building blocks were created for the PR Builder. Next Steps for supporting GitLab might be:

• We do not set 'merge_requests_events': True while creating a webhook for GitLab which does not give us access to merge request events. We have to add this for future webhook creation and also figure out a way to add this to previously created webhooks for GitLab.
  PR: Update GitLab Webhook creating to enable merge request events #6055 Auto Sync and Re-Sync for Manually Created Integrations #6071

readthedocs.org/readthedocs/oauth/services/gitlab.py

Lines 234 to 263 in e632813

	def get_webhook_data(self, repo_id, project, integration):
	"""
	Get webhook JSON data to post to the API.
	See: http://doc.gitlab.com/ce/api/projects.html#add-project-hook
	"""
	return json.dumps({
	'id': repo_id,
	'push_events': True,
	'tag_push_events': True,
	'url': 'https://{domain}{path}'.format(
	domain=settings.PRODUCTION_DOMAIN,
	path=reverse(
	'api_webhook',
	kwargs={
	'project_slug': project.slug,
	'integration_pk': integration.pk,
	},
	),
	),
	'token': integration.secret,
	# Optional
	'issues_events': False,
	'merge_requests_events': False,
	'note_events': False,
	'job_events': False,
	'pipeline_events': False,
	'wiki_events': False,
	})

• Handle incoming merge request Webhook from GitLab and creating external versions / triggering build or deleting external versions based on the webhook.
  Docs for merge request webhook event: merge-request-events. PR: PR Builder GitLab Integration #6066

• Send Build Status Report through GitLab commit status API.
  Docs for commit status API: post-the-build-status-to-a-commit PR: GitLab Build Status Reporting for PR Builder #6076

Referencing this issue as it talks about supporting GitLab for PR Builder: #2464

[stsewd]

We have to add this for future webhook creation and also figure out a way to add this to previously created webhooks for GitLab.

There are some ideas around that here #5062 (it's about github, but the same applies)

[saadmk11]

@stsewd I suppose we could do a automatic re-sync for GitLab through a Management Command.
Or maybe we can show a site notification to the users who have a GitLab project with PR Builder enabled that they need to re-sync their GitLab webhook to get the PR Builder Working.

[ericholscher]

This sounds good. I think to start we can just start setting the merge_requests on new gitlab connections. We can then worry later about how to reconnect them, after we have the rest of the code working I think.

[saadmk11]

Yeah I was thinking the same :)

[saadmk11]

Closing this as all the work for this issue has been merged.