Bug #11718

Update crunch-run for docker client lib update

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

Status:ResolvedStart date:
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:-
Target version:2017-05-24 sprint
Story points-
Velocity based estimate-

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 6 months ago

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

History

#1 Updated by Peter Amstutz 6 months ago

  • Description updated (diff)

#2 Updated by Lucas Di Pentima 6 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 6 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