Skip to content

Remove gitlab-runner user flag for shell executor

Zadkiel requested to merge zadkiel/gitlab-runner:zadkiel-main-patch-63688 into main

What does this MR do?

It removes the --user gitlab-runner when the shell executor is selected.

Why was this MR needed?

Setting runners.executor: shell makes jobs fail with su: must be suid to work properly. See the associated issue.

What's the best way to test this MR?

  • Deploy this chart with runners.executor: kubernetes and run a job with it
  • Deploy this chart with runners.executor: shell and run a job with it

What are the relevant issue numbers?

Fixes #428 (closed)

Thanks to @daniel_ulm for pointing out a solution.

Edited by Zadkiel

Merge request reports