Skip to content

WIP: Limit the size of imported GitHub repository

Yorick Peterse requested to merge gh-importer-size-limit into master

What does this MR do?

This adds a check to the GitHub importer so we reject repositories that are too large.

Why was this MR needed?

On GitLab.com we noticed many users importing large repositories, such as the Unreal Engine (30 GB). Importing very large repositories can have a negative impact on availability.

Does this MR meet the acceptance criteria?

  • Changelog entry added, if necessary
  • Tests added for this feature/bug
  • Conform by the code review guidelines
    • Has been reviewed by a Backend maintainer
    • Has been reviewed by a Database specialist
  • Conform by the merge request performance guides
  • Conform by the style guides
  • If you have multiple commits, please combine them into a few logically organized commits by squashing them
  • Internationalization required/considered
  • End-to-end tests pass (package-and-qa manual pipeline job)

TODO

  • Replace the hard coded limit with the one defined in ApplicationSetting

Merge request reports