Skip to content

Catch Timeouts in UpdateMaxSeatsUsedForGitlabComSubscriptionsWorker

Jason Goodman requested to merge catch-timeout-update-max-seats-worker into master

What does this MR do?

subscription.refresh_seat_attributes! is causing ActiveRecord::QueryCanceled errors: https://sentry.gitlab.net/gitlab/gitlabcom/issues/2620964/events/41535946/

Catch ActiveRecord::QueryCanceled errors raised in the UpdateMaxSeatsUsedForGitlabComSubscriptionsWorker. Track the errors, and continue updating remaining subscriptions.

Implements item 2 here: #329936 (comment 574893361)

Issue: #329936 (closed)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Jason Goodman

Merge request reports