An error occurred while fetching pending comments
This MR is build on the previous work on !451 (closed) by @dooferlad, to use Docker networks instead of links for inter-container communication.
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.
Container links are deprecated and discouraged by Docker
master
(if you do - rebase it please)#1042 (closed) #2511 (closed) #3173 (closed) gitlab-org/gitlab-ce#20350