Skip to content

Fix minimal CI jobs running unrelated tests

Heinrich Lee Yu requested to merge 351391-fix-ci-jobs-missing-rspec-tags into master

What does this MR do and why?

Fixes a bug where the rspec arguments from rspec_paralellized_job are not passed through to tooling/bin/parallel_rspec

This causes problems like:

  1. Geo jobs running non-Geo specs because "--tag ~geo" was not passed through.
  2. Quarantined tests being run because of the missing "--tag quarantine"

etc...

Note that this problem only happens when RSPEC_TESTS_MAPPING_ENABLED is set. So impact is limited to minimal jobs in MRs.

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

Edited by Heinrich Lee Yu

Merge request reports

Loading