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: Fam Zheng
Subject: Re: [Qemu-devel] [PATCH] Don't enable networking as a side-effect of DEBUG=1
Date: Thu, 13 Jul 2017 14:14:40 +0800
User-agent: Mutt/1.8.3 (2017-05-23)

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.

Thanks,
Fam




reply via email to

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