Skip to content

configure buildx step to use docker driver#39

Merged
zeke merged 1 commit intomainfrom
configure-buildx-step-to-use-docker-driver
Jun 17, 2024
Merged

configure buildx step to use docker driver#39
zeke merged 1 commit intomainfrom
configure-buildx-step-to-use-docker-driver

Conversation

@zeke
Copy link
Copy Markdown
Member

@zeke zeke commented Jun 17, 2024

This PR updates the docker buildx configuration to use the docker driver instead of the default docker-container driver.

@one1zero1one's thorough explanation of the issue:

Okay so I figured out the issue, the reason why it goes out to pull the image from a registry - instead of using the locally build one - is the buildx driver, which is docker-container by default - when using setup-cog.

I don't have enough context to understand if the docker-container driver is required (i.e. multi-platform features), but using docker driver in the buildx setup got me over this error and being able to successfully build and push cogs.

I spent many frustrating hours to figure this out, so I'll mention how I debugged - maybe it saves someone else time. I used mxschmitt/action-tmate@v3 action to ssh into my runner, and then used cog debug and cog build --debug to see all the steps (thx to cog developers for exposing this). There I have seen that there are couple of steps, and the last one interprets FROM r8.im/xxx/testas to literally go and pull the image, instead of using the local image. Using the right buildx driver makes it do the right thing.

Resolves these issues:

zeke added a commit to replicate/sd3-with-actions-workfow that referenced this pull request Jun 17, 2024
@zeke
Copy link
Copy Markdown
Member Author

zeke commented Jun 17, 2024

@zeke zeke merged commit 07c492a into main Jun 17, 2024
@zeke zeke deleted the configure-buildx-step-to-use-docker-driver branch June 17, 2024 23:36
@zeke
Copy link
Copy Markdown
Member Author

zeke commented Jun 17, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant