Skip to content

Remove fetch restriction

Vasilii Iakliushin requested to merge remove-fetch-restriction into master

Why is this change being made?

Related to #5990

I would like to make an experiment and remove the restriction for the depth of the fetching. According to my tests, it adds ~6 minutes to the task execution time.

I noticed that nowadays we already have a cached repository on the runner (because of the fetch strategy). Many jobs just re-initialize the repository after start but not recreating it from scratch.

The idea is to fetch the whole repository without the limit. I expect to see an immediate negative impact on CI build times. After all workers get the full copy of the repository the performance should return back to more or less current state. That's my expectation based on my understanding of our current setup.

I think testing it on production for one day will be enough to confirm or refute that. If something goes really wrong, we can always revert this MR.

Merge request reports