publicWagsHome
/
buildx
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity
966 Commits
7 Branches
41 Tags
50 MiB
Commit Graph

196 Commits (7b25e2cffc9794e7dad9ad922d1fb0748469a5d2)

Author SHA1 Message Date
Tonis Tiigi f37c253ae4 commands: provide more helpful error when —build-context is not supported 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi d77e2453da commands: rename context flag to build-context 
Avoid conflicts with docker context

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi de6b04d726 build: add support for named contexts 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 038727477c root: filter out useless debug logs from vendored packages 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi ed4103ef52 commands: build summary of warnings on build 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 9c3be32bc9 vendor: update buildkit to 539be170 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
CrazyMax 9aff9301ce
disk usage: last accessed not displayed 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 316ca972b6
bake: fix print output 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tonis Tiigi cc2a879660 commands: clean up unnecessary code 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tõnis Tiigi 89334a88a9
Merge pull request from crazy-max/fix-flags 
cli: fix flags usage
Mayeul Blanzat 72dab552b5 Add an option to buildx rm to keep the buildkitd daemon running 
Add --keep-daemon to the `rm` command option to preserve the buildkitd daemon after the buildx context is deleted.

Signed-off-by: Mayeul Blanzat <mayeul.blanzat@datadoghq.com>
CrazyMax a0a7db127c
cli: fix flags usage 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax eab0e6a8fe
cli: fix builder persistent flag 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tonis Tiigi 88d0775692 refactor accessing registry configs via drivers 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tõnis Tiigi 8afc82b427
Merge pull request from tonistiigi/project-sharedkey 
build: set local sharedkey per project basename
Tonis Tiigi 4c1621cccd store snapshot of config files on create 
Files can be reused when container needs to be booted again.

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi aa52a5a699 build: set local sharedkey per project basename 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
CrazyMax dd15969c93
build: add cgroup-parent support 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 25d2f73858
build: warning on deprecated flags 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 36a37a624e
refactor: flags lexical order 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax e150d7bdd8
add long description to root command 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 8e5595b7c7
build: add shm-size support 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax b89e2f35df
build: add ulimit support 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 9e62c9f074
Fix --driver flag usage markdown output 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tõnis Tiigi 0cea838344
Merge pull request from tonistiigi/bake-push-fix 
bake: fix using push override with output definition
Akihiro Suda 2b18a9b4a5
Merge pull request from morlay/kubeconfig-enhance 
util: support load Colon-separated KUBECONFIG
Tonis Tiigi 6fc906532b bake: fix using push override with output definition 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Morlay c83812144c util: support load Colon-separated KUBECONFIG 
Signed-off-by: Morlay <morlay.null@gmail.com>
Tõnis Tiigi b05c313204
Merge pull request from bossmc/support-quiet 
Implement `--quiet` support
CrazyMax 8a12884814
Fix flags usage markdown output 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 55a3ce606f
Built-in variable `BAKE_LOCAL_PLATFORM` 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Andy Caldwell ef3e46fd62
Move printing to stdout up to the command itself 
Signed-off-by: Andy Caldwell <andrew.caldwell@metaswitch.com>
Andy Caldwell c19c018a4c
Implement `--quiet` support 
Signed-off-by: Andy Caldwell <andrew.caldwell@metaswitch.com>
Tonis Tiigi 422ba60b04 use long-running context for client initialization 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tõnis Tiigi 04bac63745
Merge pull request from crazy-max/boostrap-cmd 
Allow booting builder after creation
CrazyMax c685e46609
bake: print default group 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 5d50bd7b43
Allow booting builder after creation 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
CrazyMax 3d1ab82dc6
Duplicated progress env var 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tõnis Tiigi 65a6955db8
Merge pull request from crazy-max/keep-buildkit-state 
Keep BuildKit state in a volume
CrazyMax 258d12b2e7
Keep BuildKit state in a volume 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tonis Tiigi 1bb425a882 bake: allow BAKE_CMD_CONTEXT builtin var 
Allows accessing the main context for bake command from bake
file that has been imported remotely.

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 9d88450118 enable opentelemetry support 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
CrazyMax 1253020b3d
Add metadata-file flag 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tõnis Tiigi 908a856079
Merge pull request from AkihiroSuda/split-flagparser 
build: split buildflags package
Akihiro Suda 399df854ea
build: split buildflags package 
Planned to be imported by nerdctl in future.

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Tonis Tiigi 328441cdc6 imagetools: fix merging JSON descriptor with old one 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Alex Couture-Beil eec843a325 include default ssh socket when given an ssh-based git url 
Signed-off-by: Alex Couture-Beil <alex@earthly.dev>
Sebastiaan van Stijn 44ddc5a02b
remove trailing whitespace in command descriptions 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Tonis Tiigi 4047bccf6c
docs: add external docs links support 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Sebastiaan van Stijn e1bbb9d8de
imagetools inspect: don't discard errors from PrintManifestList 
Looks like this function may return an error, which we currently discard.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Tonis Tiigi dba79ba223 update lint to go1.16/golangci 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
CrazyMax d05504c50f
Allow to set default progress through env var 
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Tõnis Tiigi da8c8ccaf5
Merge pull request from morlay/k8s-driver-configuration-bind 
feat: store kube config file to make buildx builder switchable
Wang 68cebffe13 feat: store kube config file to make buildx builder switchable 
Signed-off-by: Wang <morlay.null@gmail.com>
Wang Jinglei dd13e16bc7 should list real pod nodes when all drivers are kubernetes 
Signed-off-by: Wang <morlay.null@gmail.com>
Tibor Vass 381dc8fb43 driver: do not insert "platform" as driver-opt 
Addresses https://github.com/docker/setup-buildx-action/issues/45

Simple repro:
```
$ buildx create --platform linux/amd64 --use
$ buildx build - <<EOF
from scratch
EOF
```

Since https://github.com/docker/buildx/pull/370 a `platform` driver-opt was automatically inserted with the value specified by `--platform` flag on regardless of the type of driver, even though it was only used in the kubernetes driver. However, because the docker-container driver is pedantic about the options being passed, it errored out.

Another side-effect I suspect is that with the kubernetes driver it was now possible to specify the platforms in two different ways: `--driver-opt platform=...` and `--platform`.

This patch reverts completely the `platform` driver-opt and instead ensures the platforms information is passed onto the kubernetes driver via variables.

Signed-off-by: Tibor Vass <tibor@docker.com>
Tonis Tiigi cffcd57edb bake: support for remote files 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 1496ac9b55 util: simplify progress syncronization 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Wang d827f42d38 fix: buildx in k8s pod with kube client config in cluster 
Signed-off-by: Wang <morlay.null@gmail.com>
Tõnis Tiigi 517df133e3
Merge pull request from jygastaud/master 
fixes  : Replace error generated by `quiet` option by a warning.
Tonis Tiigi 378bf70d4b refactor driver auth for easier passing 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tõnis Tiigi 1ccf0bd7d8
Merge pull request from tonistiigi/buildkit-pull-creds 
docker-container: ensure credentials are passed when pulling buildkit
Tonis Tiigi ea19cf9d8d inspect: make sure to show boot error from driver 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 3b69482a2f docker-container: ensure credentials are passed when pulling buildkit image 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 4f7517115c allow builder flag to switch to default instance 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Sebastiaan van Stijn 39f1d99dcc
Remove "version" annotations from du and prune commands 
These annotations were picked up by the YAML docs generator, and shows up as
"minimum API version". I couldn't find a reference to these annotations in the
PR that added them, so thought it would be ok to remove

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Jean-Yves Gastaud 1f04ec9575 fixes 
Replace error generated by `quiet` option by a warning.

Signed-off-by: Jean-Yves Gastaud <jygastaud@gmail.com>
Ilya Dmitrichenko 4b2aab09b5
Allow users to have separate store paths 
- decouple store path from `$DOCKER_CONFIG`
- improve containerised build setup
- introduce new `$BUILDX_CONFIG` environment variable

Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
Wang Jinglei a7c704c39d feat: enhance kubernetes driver 
Signed-off-by: Wang Jinglei <morlay.null@gmail.com>
Wang Jinglei 844b901005 feat: use k8s cluster config when buildx used in k8s cluster 
Signed-off-by: Wang Jinglei <morlay.null@gmail.com>
Tõnis Tiigi 9ac5b075cf
Merge pull request from saulshanabrook/patch-1 
Increase ls and inspect timeouts
Saul Shanabrook c00c5a89e5 Increase inspect timeout from 5 to 20 seconds 
Signed-off-by: Saul Shanabrook <s.shanabrook@gmail.com>
Saul Shanabrook 14b7936c3b Increase ls timeout from 7 to 20 seconds 
Signed-off-by: Saul Shanabrook <s.shanabrook@gmail.com>
Ilya Dmitrichenko 40b41ac6e4
bake: ensure `--builder` is wired from root options 
Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
Sebastiaan van Stijn f4f58003fb
Add -f shorthand flag for prune --force 
The docker builder prune command has a shorthand `-f` flag for `--force`:

    docker builder prune --help

    Usage:	docker builder prune

    Remove build cache

    Options:
      -a, --all                  Remove all unused build cache, not just dangling ones
          --filter filter        Provide filter values (e.g. 'until=24h')
      -f, --force                Do not prompt for confirmation
          --keep-storage bytes   Amount of disk space to keep for cache

Given that `buildx` can be used as a drop-in replacement for the native build
commands, it should match the UI, and also have a shorthand flag.

This patch also updates the flag's description to be in line with the docker commandline

With this patch applied;

    buildx prune --help
    Remove build cache

    Usage:
      buildx prune [flags]

    Flags:
      -a, --all                  Remove all unused images, not just dangling ones
          --filter filter        Provide filter values (e.g. 'until=24h')
      -f, --force                Do not prompt for confirmation
      -h, --help                 help for prune
          --keep-storage bytes   Amount of disk space to keep for cache
          --verbose              Provide a more verbose output

    Global Flags:
          --builder string   Override the configured builder instance

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Tonis Tiigi 18095ee87b bake: reset no-cache and pull if not set 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tibor Vass c4d07f67e3 commands: check if flag is set instead of using flagutil.Tristate 
Fixes --pull and --no-cache without argument

Signed-off-by: Tibor Vass <tibor@docker.com>
Patrick Van Stee 4121ae50b5 Modify parsing functions and config structs to accept hcl changes 
Signed-off-by: Patrick Van Stee <patrick@vanstee.me>
Tibor Vass 09339bf500
Merge pull request from tonistiigi/platforms-print 
separate manual and automatically detected platforms
Brian Goff b2ec1d331c Add builder as a global flag. 
This allows all subcommands to use this flag.
Additionally reads the default value for the flag from the
`BUILDX_BUILDER` env var.

Precedence is:

CLI ARG > flag > env var > config file

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Brian Goff 213d3af3b0 Add option to build/bake to override instance 
This helps prevent race conditions with concurrent build invocations.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Tonis Tiigi 4804824c78 separate manual and automatically detected platforms 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tõnis Tiigi d89e3f3014
Merge pull request from tonistiigi/prune 
add prune and du commands
Tibor Vass 2a257a8252 bake: allow overriding no-cache and pull per target via --set 
Signed-off-by: Tibor Vass <tibor@docker.com>
Tibor Vass 417f52e001 bake: add --load and --push shorthands for --set 
Signed-off-by: Tibor Vass <tibor@docker.com>
Tibor Vass 2bca8fa677 bake: allow pattern matching for target names in --set 
Although bake is for running multiple targets, --set required a single
target name for overriding a property. This change allows matching
multiple targets for overrides.

Signed-off-by: Tibor Vass <tibor@docker.com>
Tonis Tiigi d52f5db6ba commands: add du command 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 66672b4052 commands: add prune command 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Brian Goff 5e4444823c build: only use env for args if set 
When following this pattern:

  buildx build --arg FOO

Where we want to pull `FOO` from env, currently we always set `FOO`
regardless if the `FOO` env var is even set.

This change makes it so that `FOO` would only be set if it has been set
in the env (even if it is set to empty).

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Akihiro Suda c6f8de90aa kubernetes: show Kubernetes Pods as buildx "Nodes" in `docker buildx inspect` 
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Akihiro Suda 6b65b0c982 new driver: kubernetes 
Tested with `kind` and GKE.

Note: "nodes" shown in `docker buildx ls` are unrelated to Kubernetes "nodes".
Probably buildx should come up with an alternative term.

Usage:

  $ kind create cluster
  $ export KUBECONFIG="$(kind get kubeconfig-path --name="kind")"

  $ docker buildx create --driver kubernetes --driver-opt replicas=3 --use
  $ docker buildx build -t foo --load .

`--load` loads the image into the local Docker.

Driver opts:

  - `image=IMAGE` - Sets the container image to be used for running buildkit.
  - `namespace=NS` - Sets the Kubernetes namespace. Defaults to the current namespace.
  - `replicas=N` - Sets the number of `Pod` replicas. Defaults to 1.
  - `rootless=(true|false)` - Run the container as a non-root user without `securityContext.privileged`. Defaults to false.
  - `loadbalance=(sticky|random)` - Load-balancing strategy. If set to "sticky", the pod is chosen using the hash of the context path. Defaults to "sticky"

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Tonis Tiigi eb1aabe9e3 imagetools: avoid printing newline on raw mode 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 75b80c277f driver: allow setting driver opts 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tibor Vass 096d1befc9
Merge pull request from tonistiigi/entitlements 
build: add allowed entitlements
Tonis Tiigi 8ed8795268 driver: allow setting buildkit config file 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Co-Authored-By: Tibor Vass <tiborvass@users.noreply.github.com>
Tibor Vass 8b2171f78a bake: honor --no-cache and --pull 
Signed-off-by: Tibor Vass <tibor@docker.com>
Tonis Tiigi 43150ef849 build: load default build args from env 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi 6b81b0bed6 build: add allowed entitlements 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Tonis Tiigi f0af89a204 driver: allow configuring buildkitd flags 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Akihiro Suda eee78c6c10 Put driver names to create --help 
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>