Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Upgrade docker/docker #1685

Merged
merged 2 commits into from Jun 22, 2021
Merged

Upgrade docker/docker #1685

merged 2 commits into from Jun 22, 2021

Conversation

izaaklauer
Copy link
Contributor

@izaaklauer izaaklauer commented Jun 22, 2021

Fixes #1684.

It's a bit of a mystery to me why running go get github.com/docker/docker; go mod tidy resulted in this many dependency changes, including a bbolt upgrade.

Fixes #1684.

It's a bit of a mystery to me why running `go get github.com/docker/docker; go mod tidy` resulted in this big of a dependency chain, including a bbolt upgrade.
Copy link
Contributor

@mitchellh mitchellh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I'd tag this for backport.

Copy link
Contributor

@krantzinator krantzinator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So did we downgrade docker at some point and get the issue? Or was the issue introduced in one upgrade of docker and then resolved again in a subsequent upgrade of docker?
Fun fun.

@evanphx
Copy link
Contributor

evanphx commented Jun 22, 2021

I think the previous upgrade of docker introduce the issue, and then this upgrade fixes it. Any idea which change fixes it?

@izaaklauer
Copy link
Contributor Author

I think the previous upgrade of docker introduce the issue, and then this upgrade fixes it. Any idea which change fixes it?

This is what fixes it: moby/moby#41667

I actually found and made that fix myself locally, and then was disappointed when I rushed to make the docker PR and realized we were behind and someone had beaten me to it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: build never falls back to img
4 participants