What does this MR do?
I added a function
connectToBuildNetwork to connect the created containers to a special build network, that is removed after the job finishes.
Question: do we want to support both legacy links and network based on the API version of Docker? See here for the conditional switch between the two.
I saw this commit 226ecfe0, so if it is merged we can safely deprecate links, given that with that minimum API of Docker we assume to have networking available.
Why was this MR needed?
Container links are deprecated and discouraged by Docker
Are there points in the code the reviewer needs to double check?
Does this MR meet the acceptance criteria?
Added for this feature/bug
All builds are passing
Branch has no merge conflicts with
master(if you do - rebase it please)