qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] Don't enable networking as a side-effect of DEB


From: Alex Bennée
Subject: Re: [Qemu-devel] [PATCH] Don't enable networking as a side-effect of DEBUG=1
Date: Thu, 13 Jul 2017 09:46:30 +0100
User-agent: mu4e 0.9.19; emacs 25.2.50.3

Fam Zheng <address@hidden> writes:

> On Wed, 07/12 17:25, Daniel P. Berrange wrote:
>> When trying to debug problems with tests it is natural to set
>> DEBUG=1 when starting the docker environment. Unfortunately
>> this has a side-effect of enabling an eth0 network interface
>> in the container, which changes the operating environment of
>> the test suite. IOW tests with fail may suddenly start
>> working again if DEBUG=1 is set, due to changed network setup.
>
> Makes sense.
>
>>
>> Add a separate NETWORK=1 option to allow enablement of
>> networking separately from DEBUG=1, since common debugging
>> tasks probably don't require networking anyway.
>
> Not uncommon because fiddling with the package manager is often needed when
> working on the test coverage. But that doesn't mean it's bad to control 
> network
> explicitly.
>
>>
>> Signed-off-by: Daniel P. Berrange <address@hidden>
>> ---
>>  tests/docker/Makefile.include | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
>> index 037cb9e..a8c4b82 100644
>> --- a/tests/docker/Makefile.include
>> +++ b/tests/docker/Makefile.include
>> @@ -106,6 +106,7 @@ docker:
>>      @echo '                         (default is 1)'
>>      @echo '    DEBUG=1              Stop and drop to shell in the created 
>> container'
>>      @echo '                         before running the command.'
>> +    @echo '    NETWORK=1            Enable eth0 virtual network interface.'
>
> As pointed out by Philippe, I'd just document it as "enable network".
>
>>      @echo '    NOUSER               Define to disable adding current user 
>> to containers passwd.'
>>      @echo '    NOCACHE=1            Ignore cache when build images.'
>>      @echo '    EXECUTABLE=<path>    Include executable in image.'
>> @@ -132,7 +133,8 @@ docker-run: docker-qemu-src
>>              $(SRC_PATH)/tests/docker/docker.py run                  \
>>                      $(if $(NOUSER),,-u $(shell id -u)) -t           \
>>                      $(if $V,,--rm)                                  \
>> -                    $(if $(DEBUG),-i,--net=none)                    \
>> +                    $(if $(DEBUG),-i,)                              \
>> +                    $(if $(NETWORK),,--net=none)                    \
>>                      -e TARGET_LIST=$(TARGET_LIST)                   \
>>                      -e EXTRA_CONFIGURE_OPTS="$(EXTRA_CONFIGURE_OPTS)" \
>>                      -e V=$V -e J=$J -e DEBUG=$(DEBUG)               \
>> --
>> 2.9.4
>>
>
> If others don't come up with objections, I'll apply this.

Acked-by: Alex Bennée <address@hidden>


--
Alex Bennée



reply via email to

[Prev in Thread] Current Thread [Next in Thread]