Skip to content

Fix VSA FF scope to groups

Adam Hegyi requested to merge 344802-fix-vsa-summary-endpoint-ff-scoping into master

What does this MR do and why?

We enabled the use_vsa_aggregated_tables feature flag for gitlab-org and it works pretty well for now. We've noticed a small problem where the feature flag does not "apply" for the summary endpoint.

It turns out we didn't pass the group to the RequestParams instance thus the enabled FF state was derived from the global FF state (disabled).

Note: This only affects the FF check, the endpoint works as expected and the queries are scoped to the current group.

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

Merge request reports