Skip to content

ci: Fix broken master by not reading GITLAB_ENV

Rémy Coutable requested to merge fix-broken-master into master

What does this MR do and why?

This fixes a broken master problem that started after part of the Runner fleet was updated to 16.6.0~beta.105.gd2263193: gitlab-org/quality/engineering-productivity/master-broken-incidents#4168 (comment 1646281188).

This removes the ability to read from GITLAB_ENV and always use Rails.env instead.

These variables aren't documented at all so I think this shouldn't create any problem.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Rémy Coutable

Merge request reports