Priyusha Ch commented on merge request !2109 at GitLab.org / Developer Relations / DevRel Engineering / contributors-gitlab-com

Priyusha Ch
2026-04-21T15:18:18Z

Hey @leetickett-gitlab , can you please review the MR.

When I'm initiating GitLab Duo in comments for review, its saying - I have encountered some problems while I was reviewing. Please try again later.

So I did initiate in GitLab Duo Chat to review and got below response.

Great news — the MR is in excellent shape. Here's the full current status:

!2109 (merged) Review Summary

✅ Pipeline: passing ✅ All discussions: resolved (blocking_discussions_resolved: true) ✅ Status: mergeable

gitlab_graphql_service.rb — looks good

graphql_api_url correctly uses ContributorPlatformSettings::GITLAB_GRAPHQL_URL (the fix was applied)

RetryableServerError, RETRYABLE_EXCEPTIONS, retryable_error?, validate_response!, and timeout_or_expired_message? are all correctly implemented

Selective retry logic (raise unless retryable_error?(e)) is in place

gitlab_graphql_service_spec.rb — looks good

'fails' shared example uses 'timeout error' message to correctly trigger retry

All new contexts are covered: non-retryable StandardError, non-Hash 200 body, GraphQL timeout retry with sleep.twice assertion, network-level errors, non-retryable errors, 5xx retry, 4xx immediate raise

No modifications required. The MR is ready for maintainer approval and merge. The only thing left is for @leetickett-gitlab to review and approve it.