Re: [Xen-devel] [OSSTEST PATCH] ts-openstack-deploy: Set http proxy

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

On Tue, Apr 25, 2017 at 04:14:32PM +0100, Ian Jackson wrote:
> This allows ./stack.sh to access the global internet.
> 
> CC: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> ---
>  ts-openstack-deploy | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/ts-openstack-deploy b/ts-openstack-deploy
> index a1c974f..d4c041d 100755
> --- a/ts-openstack-deploy
> +++ b/ts-openstack-deploy
> @@ -137,8 +137,11 @@ END
>  }
>  
>  sub deploy() {
> +    my $httpproxy = http_proxy_envsettings($ho);
> +
>      target_cmd($ho, <<END, 1800);
>          set -e
> +        $httpproxy
>          cd $builddir/devstack
>          ./stack.sh
>  END

The patch itself looks good, but I think http_proxy_envsettings only set
http_proxy, twice, but does not set https_proxy.

For this patch:
Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

-- 
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel