Skip to content

Conversation

@realbigsean
Copy link
Member

Issue Addressed

N/A

Proposed Changes

  • hardcode ubuntu-18.04 -- I don't think this was causing us issues, but github actions is in the process of migrating ubuntu-latest from Ubuntu 18 -> 20.. so just in case
  • different source of emulation dependencies -> https://github.com/tonistiigi/binfmt
    • this one is explicitly referenced in the buildx github docs
  • install emulation dependencies and run docker buildx in the same run command
  • enable buildx with DOCKER_CLI_EXPERIMENTAL: enabled rather than re-building it

Additional Info

N/A

@realbigsean realbigsean added the ready-for-review The code is ready for review label Dec 10, 2020
@michaelsproul
Copy link
Member

I can't fault this, let's do it!

bors r+

bors bot pushed a commit that referenced this pull request Dec 11, 2020
## Issue Addressed

N/A

## Proposed Changes

- hardcode `ubuntu-18.04` -- I don't think this was causing us issues, but github actions is in the process of migrating `ubuntu-latest` from Ubuntu 18 -> 20.. so just in case
- different source of emulation dependencies -> https://github.com/tonistiigi/binfmt 
  - this one is explicitly referenced in the `buildx` github docs
- install emulation dependencies and run `docker buildx` in the same `run` command
- enable `buildx` with  `DOCKER_CLI_EXPERIMENTAL: enabled` rather than re-building it

## Additional Info

N/A


Co-authored-by: realbigsean <seananderson33@gmail.com>
@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Dec 11, 2020
@bors
Copy link

bors bot commented Dec 11, 2020

@bors bors bot changed the title Improve docker auto builds [Merged by Bors] - Improve docker auto builds Dec 11, 2020
@bors bors bot closed this Dec 11, 2020
@realbigsean realbigsean deleted the improve-docker-auto-builds branch November 21, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants