[v2] Various docker fixes

[Qemu-devel] [PATCH for 2.10 v2 0/6] Various docker fixes

Posted by Alex Bennée 8 years, 3 months ago

Hi,

Phillipe pointed out a few anomalies with the travis.docker image
during review which has led to a couple more patches. As they are bug
fixes I didn't race to get a pull-req out today for hard-freeze. I
will roll a pull-req by the end of the week if there are no objections
as I'll be away for the rc1/2 cycles.

Alex Bennée (6):
  docker: ensure NOUSER for travis images
  docker: fix dirty/stash detection on some systems
  docker: ignore submodules when checking diff
  docker: docker.py make --no-cache skip checksum test
  docker: don't install device-tree-compiler build-deps in travis.docker
  docker: reduce noise when building travis.docker

 tests/docker/Makefile.include          | 3 ++-
 tests/docker/docker.py                 | 3 ++-
 tests/docker/dockerfiles/travis.docker | 6 ++++--
 3 files changed, 8 insertions(+), 4 deletions(-)

-- 
2.13.0

Re: [Qemu-devel] [PATCH for 2.10 v2 0/6] Various docker fixes

Posted by Fam Zheng 8 years, 2 months ago

On Tue, 07/25 14:34, Alex Bennée wrote:
> Hi,
> 
> Phillipe pointed out a few anomalies with the travis.docker image
> during review which has led to a couple more patches. As they are bug
> fixes I didn't race to get a pull-req out today for hard-freeze. I
> will roll a pull-req by the end of the week if there are no objections
> as I'll be away for the rc1/2 cycles.
> 
> Alex Bennée (6):
>   docker: ensure NOUSER for travis images
>   docker: fix dirty/stash detection on some systems
>   docker: ignore submodules when checking diff
>   docker: docker.py make --no-cache skip checksum test
>   docker: don't install device-tree-compiler build-deps in travis.docker
>   docker: reduce noise when building travis.docker

Thanks, I've queued patches 1, 4, 5, 6 for 2.11. I intend to drop the code
touched by 2 and 3 with a dedicated script:

https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06349.html
https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06358.html

(It would be great if you can give some test/review there :)

Fam

> 
>  tests/docker/Makefile.include          | 3 ++-
>  tests/docker/docker.py                 | 3 ++-
>  tests/docker/dockerfiles/travis.docker | 6 ++++--
>  3 files changed, 8 insertions(+), 4 deletions(-)
> 
> -- 
> 2.13.0
>

Re: [Qemu-devel] [PATCH for 2.10 v2 0/6] Various docker fixes

Posted by Alex Bennée 8 years, 2 months ago

Fam Zheng <famz@redhat.com> writes:

> On Tue, 07/25 14:34, Alex Bennée wrote:
>> Hi,
>>
>> Phillipe pointed out a few anomalies with the travis.docker image
>> during review which has led to a couple more patches. As they are bug
>> fixes I didn't race to get a pull-req out today for hard-freeze. I
>> will roll a pull-req by the end of the week if there are no objections
>> as I'll be away for the rc1/2 cycles.
>>
>> Alex Bennée (6):
>>   docker: ensure NOUSER for travis images
>>   docker: fix dirty/stash detection on some systems
>>   docker: ignore submodules when checking diff
>>   docker: docker.py make --no-cache skip checksum test
>>   docker: don't install device-tree-compiler build-deps in travis.docker
>>   docker: reduce noise when building travis.docker
>
> Thanks, I've queued patches 1, 4, 5, 6 for 2.11. I intend to drop the code
> touched by 2 and 3 with a dedicated script:
>
> https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06349.html
> https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg06358.html
>
> (It would be great if you can give some test/review there :)

Cool - I shall have a look.

>
> Fam
>
>>
>>  tests/docker/Makefile.include          | 3 ++-
>>  tests/docker/docker.py                 | 3 ++-
>>  tests/docker/dockerfiles/travis.docker | 6 ++++--
>>  3 files changed, 8 insertions(+), 4 deletions(-)
>>
>> --
>> 2.13.0
>>


--
Alex Bennée