Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Allow copa to use buildkit from dockerd #233

Merged
merged 3 commits into from
Aug 15, 2023
Merged

feat: Allow copa to use buildkit from dockerd #233

merged 3 commits into from
Aug 15, 2023

Commits on Aug 14, 2023

  1. feat: Support buildkit client connection helpers 

    - Adds all the built-in connection helpers from upstream buildkit (e.g.
  `docker-container://`, `ssh://`, `kubepod://`
- Add connection helpers for:
  - dockerd's built-in buildkit (`docker://`)
  - buildx configured buildkit instances (`buildx://`)
- When addr is unset try to find a suitable buildkit instance
  - Priority: docker -> buildx -> default buildkit addr

For the buildx support, currently only buildx instances over
docker-containers are supported. Support other buildx drivers is
possible (e.g. the kubernetes driver), but this is a lot more effort
that can come later.

Signed-off-by: Brian Goff <[email protected]>
    @cpuguy83
    cpuguy83 committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    285bafe View commit details
    Browse the repository at this point in the history

  2. Add test matrix to include different buildkit connectors 

    This is just a way to test all the connection helpers using the existing
test harness.
Ideally the patch tests are refactored into use go and we can more
easily manage mutliple parallel environments from one runner.

Signed-off-by: Brian Goff <[email protected]>
    @cpuguy83
    cpuguy83 committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    50b9bf9 View commit details
    Browse the repository at this point in the history

  3. Move patch tests to go integration tests 

    Co-authored-by: Sertaç Özercan <[email protected]>
Signed-off-by: Brian Goff <[email protected]>
    @cpuguy83 @sozercan
    cpuguy83 and sozercan committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    57c66e8 View commit details
    Browse the repository at this point in the history