Skip to content

Start Praefect using Gitaly container's process wrapper

Balasankar 'Balu' C requested to merge praefect-gitaly-wrapper into master

What does this MR do?

Instead of using a ConfigMap hacked in via args to run Praefect, make gitaly's process-wrapper handle both Gitaly and Praefect services, since we are using the same Docker image for both Gitaly and Praefect.

The CNG part to add the support to process-wrapper script of Gitaly is being done in gitlab-org/build/CNG!729 (merged). This MR removes the configmap/scripts and sets necessary environment variables.

Related issues

#2434 (closed)

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion.

Required

  • Merge Request Title and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com

Expected (please provide an explanation if not completing)

  • Test plan indicating conditions for success has been posted and passes
  • Documentation created/updated
  • Tests added
  • Integration tests added to GitLab QA
  • Equivalent MR/issue for omnibus-gitlab opened
Edited by Jason Plum

Merge request reports