Remove flaky tests from blocking suite
What does this MR do and why?
Removes the tests in qa/specs/features/browser_ui/4_verify/ci_variable/ui_variable_non_inheritable_when_forward_pipeline_variables_false_spec.rb
from the :blocking
suite because their flakiness can (and has) caused incidents blocking master.
See https://gitlab.com/gitlab-org/gitlab/-/issues/470904#note_2013076687
It's fast-quarantined in gitlab-org/quality/engineering-productivity/fast-quarantine!236 (merged) so should be removed from fast-quarantine once this is merged.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Edited by Mark Lapierre