Re: [Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-17 Thread Debarshi Ray
On Wed, Jul 10, 2019 at 10:40 AM Marc-André Lureau wrote: > I am not very familiar with podman or docker, so I am not able to tell > you why docker does work by default. @Debarshi Ray might know, as he > helped me finding a workaround. You need to mention the UID mapping via --uidmap arguments (

Re: [Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-17 Thread Debarshi Ray
Hey, Sorry for the late response. I was on vacation and away from my keyboard. On Tue, Jul 9, 2019 at 9:44 PM Marc-André Lureau wrote: > With current podman, we have to use a uidmap trick in order to be able > to rw-share the ccache directory with the container user. > > With a user 1000, the de

Re: [Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-11 Thread Alex Bennée
Paolo Bonzini writes: > On 10/07/19 10:39, Marc-André Lureau wrote: >>> Why not do this in docker.py (either as part of patch 1 or separately)? >>> Also, can you document in a comment why this is not needed with docker? >> >> Doing it in docker.py would probably mean parsing and tweaking >> ar

Re: [Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-10 Thread Paolo Bonzini
On 10/07/19 10:39, Marc-André Lureau wrote: >> Why not do this in docker.py (either as part of patch 1 or separately)? >> Also, can you document in a comment why this is not needed with docker? > > Doing it in docker.py would probably mean parsing and tweaking > arguments given to Docker.run(). Si

Re: [Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-10 Thread Marc-André Lureau
Hi On Wed, Jul 10, 2019 at 12:27 PM Paolo Bonzini wrote: > > On 09/07/19 21:43, Marc-André Lureau wrote: > > With current podman, we have to use a uidmap trick in order to be able > > to rw-share the ccache directory with the container user. > > > > With a user 1000, the default mapping is: > > 1

Re: [Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-10 Thread Paolo Bonzini
On 09/07/19 21:43, Marc-André Lureau wrote: > With current podman, we have to use a uidmap trick in order to be able > to rw-share the ccache directory with the container user. > > With a user 1000, the default mapping is: > 1000 (host) -> 0 (container). Why not do this in docker.py (either as pa

[Qemu-devel] [PATCH v2 2/5] tests/docker: add podman support

2019-07-09 Thread Marc-André Lureau
Allow to specify the container engine to run with ENGINE variable. By default, ENGINE=auto and will select either podman or docker. With current podman, we have to use a uidmap trick in order to be able to rw-share the ccache directory with the container user. With a user 1000, the default mappi