Skip to content

Resolve "Add a test for `service.variables`"

Albert requested to merge 431993-add-a-test-for-service-variables into master

What does this MR do and why?

Add test gap on service config entry.

There was an incident gitlab-com/gl-infra/production#17143 (closed). The incident was caused by a change that inadvertently dropped service.variables from the #values method.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Related to #431993 (closed)

Edited by Albert

Merge request reports