Bug #11718

Update crunch-run for docker client lib update

Added by Peter Amstutz 11 months ago. Updated 11 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
-
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Story points:
-

Description

The function signature for Docker.ContainerWait has changed, this is breaking crunch-run.

In addition, github.com/docker/docker now redirects to github.com/moby/moby

Associated revisions

Revision f0e917d2
Added by Peter Amstutz 11 months ago

Merge branch '11718-crunch-run-docker-wait' closes #11718

History

#1 Updated by Peter Amstutz 11 months ago

  • Description updated (diff)

#2 Updated by Lucas Di Pentima 11 months ago

Reviewed 9d33e0c4f
Jenkins tests ran ok: https://ci.curoverse.com/job/developer-run-tests/288/

There's no reference to github.com/moby/moby imports, and on docker's documentation it seems that it isn't updated yet, I suppose the redirect will be kept for a long time.

LGTM.

#3 Updated by Peter Amstutz 11 months ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:f0e917d2305c66cb7120125aaf6592ce2b9e3855.

Also available in: Atom PDF