Loading
Drop the license fallback and allow_failure from cng-secrets-manager
What does this MR do and why?
The shared QA_EE_LICENSE CI variable has been rotated to a license that carries the secrets_manager add-on (see https://gitlab.com/gitlab-org/gitlab/-/work_items/628661#note_3835761559). The full e2e suites passed against it in !255365, so the temporary scaffolding on the cng-secrets-manager job is no longer needed.
This MR:
- Removes
allow_failure: truefromcng-secrets-managerin.gitlab/ci/test-on-cng/main.gitlab-ci.yml, so the job blocks again when the Secrets Manager specs fail. - Removes the
before_scriptoverride that exportedQA_EE_LICENSE="${QA_EE_LICENSE_SECRETS_MANAGER:-$QA_EE_LICENSE}". TheQA_EE_LICENSE_SECRETS_MANAGERproject variable has been deleted, so the job now inherits the standard.cng-testbefore_script. - Updates the error message in
qa/qa/ee/support/helpers/secrets_management/secrets_manager_helper.rbso it points atQA_EE_LICENSEinstead of the removed variable.
References
- Issue: https://gitlab.com/gitlab-org/gitlab/-/work_items/628661
- Test MR that validated the rotated license: !255365
- MR that introduced the fallback: !255043 (merged)
How to set up and validate locally
Not applicable, this is a CI configuration change. Confirm the cng-secrets-manager job in the e2e:test-on-cng child pipeline of this MR passes without allow_failure.
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.