Skip to content

Removes repository_import jobs from catchall shard

John Skarbek requested to merge jts/remove-repImports-from-catchall into master

What does this MR do?

ea9c53b4 - Removes repository_import jobs from catchall shard

  • Now that the new imports shard is up and running, we can remove this worker from the catchall shard

474e0ed0 - Bumps pod counts for imports

  • The imports shard was over 80% saturated with the catchall shard in play
  • Doubling the number of Pods to minimize potential impact
  • Currently I'm not sure of a way to determine overall saturation when other workers are running on a shard. At the moment, the math told me the catchall fleet was less than 1% saturated for ONLY the repository_import worker

Addresses: gitlab-org/gitlab#332616 (closed)

There is risk that if the new imports shard lack capacity, jobs will stay queued for extended periods of time. This should be closely monitored upon initial rollout.

Author Check-list

Please read the Contributing document and once you do, complete the following:

  • Assign to the correct reviewer per the contributing document
  • Apply the correct metadata per the contributing document

Reviewer Check-list

  • Reviewed the diff jobs to confirm changes are as expected
  • No changes shown in the diffs not associated with this MR - This may require a rebase or further investigation
Edited by Craig Miskell

Merge request reports