moby/api/server
Brian Goff 88c0271605 Don't set default platform on container create
This fixes a regression based on expectations of the runtime:

```
docker pull arm32v7/alpine
docker run arm32v7/alpine
```

Without this change, the `docker run` will fail due to platform
matching on non-arm32v7 systems, even though the image could run
(assuming the system is setup correctly).

This also emits a warning to make sure that the user is aware that a
platform that does not match the default platform of the system is being
run, for the cases like:

```
docker pull --platform armhf busybox
docker run busybox
```

Not typically an issue if the requests are done together like that, but
if the image was already there and someone did `docker run` without an
explicit `--platform`, they may very well be expecting to run a native
version of the image instead of the armhf one.

This warning does add some extra noise in the case of platform specific
images being run, such as `arm32v7/alpine`, but this can be supressed by
explicitly setting the platform.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
2020-10-20 20:17:23 +00:00
..
backend/build API: add "prune" events 2020-07-28 12:41:14 +02:00
httputils api/server: use constants for http methods 2019-10-13 17:30:19 +02:00
middleware bump gotest.tools v3.0.1 for compatibility with Go 1.14 2020-02-11 00:06:42 +01:00
router Don't set default platform on container create 2020-10-20 20:17:23 +00:00
middleware.go Add canonical import comment 2018-02-05 16:51:57 -05:00
server.go Remove unused router swapper 2019-09-04 13:50:03 -07:00
server_test.go api/server: use constants for http methods 2019-10-13 17:30:19 +02:00