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.per...@citrix.com>
> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com>
> ---
> 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.per...@citrix.com>
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
