Skip to content

Added GIT_CLEAN variable support to skip git clean

Pier-Luc Gagnon requested to merge (removed):master into master

What does this MR do?

It adds support for setting GIT_CLEAN variable.

Why was this MR needed?

Reusing generated files between builds can significantly speed up the build process. Caching is one solution but can be very slow on large directories/files. Using GIT_STRATEGY=none can be used as a workaround, but requires the user to do their own clone/fetch/checkout/etc. which is inconvenient and can lead to issues (for instance the artifacts are downloaded before the gitlab-ci.yml is executed.

Are there points in the code the reviewer needs to double check?

Not cleaning might have consequences on checkout or other git operations further down the pipeline.

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

https://forum.gitlab.com/t/disable-git-clean-between-jobs/7494 (not actually an issue per say)

Other

Similar to: !196 (closed) but not runner-level and more similar to how GIT_CHECKOUT is handled.

Edited by Pier-Luc Gagnon

Merge request reports