Skip to content

Don't fail check_if_qa_only_spec_changes if tmp/changed_files.txt missing

Rémy Coutable requested to merge 347202-package-and-qa-fails-consistently-in-master into master

What does this MR do and why?

In master pipelines, tmp/changed_files.txt doesn't exist as we're not in an MR context.

For these pipelines, the tooling/bin/qa/check_if_qa_only_spec_changes check should just early return and not write any file, and the rest of the job should run normally.

Validation: package-and-qa didn't fail even if the tmp/changed_files.txt file doesn't exist: https://gitlab.com/gitlab-org/gitlab/-/jobs/1853390818

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 #347202 (closed)

Edited by Rémy Coutable

Merge request reports