Skip to content

Add missing check for analytics page

What does this MR do?

This adds a missing if check in case the graphql_pipeline_analytics feature flag is enabled, avoiding sending double the requests for getting the chart counts inside the Project -> Analytics -> CI/CD page.

In the aforementioned page currently, those queries take a significant amount of time, causing said queries to time out and causing a 500 error, even with the feature flag turned on. Leaving those queries out would allow for the new GraphQL type and query to request the data more efficiently. The GraphQL related code was introduced here: !49384 (merged)

The original MR where this feature flag was introduced

!48267 (merged)

The rollout issue

#290153 (closed)

Screenshots (strongly suggested)

N/A

Does this MR meet the acceptance criteria?

Conformity

Edited by Jose Ivan Vargas

Merge request reports