Skip to content

WIP: Mount user volumes to services

Kamil Trzcińśki requested to merge mount-user-volumes-to-services into master

This is a general Merge Request template. Consider to choose a template from the list above if it will match your case more.

What does this MR do?

This fixes a bug when docker executor is used, and we do use permament volumes = ["/cache:/cache", "/builds:/builds"].

Then, if using docker:dind we cannot access /builds folder, as it is not mounted to the service container.

This changes the order of operation to make this possible.

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?

cc @nolith

Merge request reports