Add API Version Negotiation to Docker client

[rohitkg98]

Closes #154

[codecov-commenter]

Codecov Report

Merging #155 (713f6ee) into master (fdc2a23) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
+ Coverage   82.65%   82.67%   +0.01%     
==========================================
  Files          46       46              
  Lines        1810     1812       +2     
==========================================
+ Hits         1496     1498       +2     
  Misses        163      163              
  Partials      151      151              

Impacted Files	Coverage Δ
docker.go	88.60% <100.00%> (+0.14%)	⬆️
gnomock.go	81.88% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fdc2a23...713f6ee. Read the comment docs.

[orlangure]

Hi @rohitkg98, thank for your contribution!

It appears that docker client has a data race when version negotiation is used, so I submitted moby/moby#42379 to fix it, and to prevent test failure in gnomock (we run most tests with -race flag on).

While I wait for resolution of the mentioned PR, I'll try to figure out how to prevent data race locally, in gnomock, so your change doesn't break any tests (even though the failure is in another package).

Sorry for not merging it sooner.

[orlangure]

Hi @rohitkg98,
I pushed 394c35b into your original branch. It fixes the data race in short term. What do you think?

If it looks OK, please rebase from master and I'll merge your fix. Thanks!

[rohitkg98]

Looks good! Rebased and ready to merge! Thanks for promptly addressing the issue!