feat: TEST defaults to false when not in dry-run mode

Related to gitlab-com/gl-infra/delivery#21271 (closed).

This implements one possible solution to the above issue.

When the option --dry-run is not passed, currently the value for TEST is empty. When this is used within GitLab CI to trigger another pipeline, the value of that variable is empty and can not be overwritten by downstream projects that are using this variable. This is confusing. If TEST is truly a boolean value, then we should at least always pass either true or false, so that the value is never empty.

Otherwise, there is a case where setting TEST to true works, but there is no way to set it to false.

Note

This is a draft MR for the consideration of reviewers at the moment. If reviewers agree with the general approach, I will implement tests for this behavior.

Edited by Siddharth Kannan

Merge request reports

Loading