Skip to content

Instruct review roulette to flag graphql changes for database review

Gregory Havenga requested to merge ghavenga-require-db-review-graphql into master

What does this MR do and why?

It has been noted that the merge of !109661 (merged) resulted in an incident where our E2E tests began detecting a failure in our production and canary environments.

Adding the graphql/resolvers directory to the review roulette database required review set should help us catch query changes with possible failures a little more readily, and shift some of our failures left.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports