Skip to content

#2 - Default shell executor, allow configuration through several executor options

Drane, Lance requested to merge 2-flexible-executor into master

Closes #2 (closed)

Intention is to make this module extremely extensible, so we don't have to do the work for INTERSECT later if we decide to switch the executor.

It would probably be better to validate via Terraform instead of a bash script, but I couldn't think of a way to do so. My general approach is to set several executor-specific default variables to an empty string through Terraform, then check to see if they are not an empty string in Bash.

References:

Gitlab Runner registration code: https://gitlab.com/gitlab-org/gitlab-runner/-/blob/main/commands/register.go (pretty easy to parse from the askExecutorOptions() function) . My general approach was to CTRL+F ask( , which seems to correspond with the required CLI arguments for each executor.

Terraform variables docs: https://www.terraform.io/docs/language/values/variables.html

Merge request reports