diff mbox series

[v1,05/17] tests/vm: send proxy environment variables over ssh

Message ID 20190619194021.8240-6-alex.bennee@linaro.org
State Superseded
Headers show
Series testing/next (Travis fixes, more tests/vm) | expand

Commit Message

Alex Bennée June 19, 2019, 7:40 p.m. UTC
From: Gerd Hoffmann <kraxel@redhat.com>


Packages are fetched via proxy that way, if configured on the host.
That might be required to pass firewalls, and it allows to route
package downloads through a caching proxy server.

Needs AcceptEnv setup in sshd_config on the guest side to work.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Tested-by: Thomas Huth <thuth@redhat.com>

Message-Id: <20190617043858.8290-2-kraxel@redhat.com>
---
 tests/vm/basevm.py | 9 +++++++++
 1 file changed, 9 insertions(+)

-- 
2.20.1

Comments

Philippe Mathieu-Daudé June 20, 2019, 10:38 a.m. UTC | #1
On 6/19/19 9:40 PM, Alex Bennée wrote:
> From: Gerd Hoffmann <kraxel@redhat.com>

> 

> Packages are fetched via proxy that way, if configured on the host.

> That might be required to pass firewalls, and it allows to route

> package downloads through a caching proxy server.

> 

> Needs AcceptEnv setup in sshd_config on the guest side to work.

> 

> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> Tested-by: Thomas Huth <thuth@redhat.com>


Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>


> Message-Id: <20190617043858.8290-2-kraxel@redhat.com>


Signed-off-by: Alex Bennée <alex.bennee@linaro.org>


> ---

>  tests/vm/basevm.py | 9 +++++++++

>  1 file changed, 9 insertions(+)

> 

> diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py

> index 4847549592..5e30bac661 100755

> --- a/tests/vm/basevm.py

> +++ b/tests/vm/basevm.py

> @@ -38,6 +38,13 @@ class BaseVM(object):

>      GUEST_PASS = "qemupass"

>      ROOT_PASS = "qemupass"

>  

> +    envvars = [

> +        "https_proxy",

> +        "http_proxy",

> +        "ftp_proxy",

> +        "no_proxy",

> +    ]

> +

>      # The script to run in the guest that builds QEMU

>      BUILD_SCRIPT = ""

>      # The guest name, to be overridden by subclasses

> @@ -106,6 +113,8 @@ class BaseVM(object):

>                     "-o", "UserKnownHostsFile=" + os.devnull,

>                     "-o", "ConnectTimeout=1",

>                     "-p", self.ssh_port, "-i", self._ssh_key_file]

> +        for var in self.envvars:

> +            ssh_cmd += ['-o', "SendEnv=%s" % var ]

>          if interactive:

>              ssh_cmd += ['-t']

>          assert not isinstance(cmd, str)

>
diff mbox series

Patch

diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py
index 4847549592..5e30bac661 100755
--- a/tests/vm/basevm.py
+++ b/tests/vm/basevm.py
@@ -38,6 +38,13 @@  class BaseVM(object):
     GUEST_PASS = "qemupass"
     ROOT_PASS = "qemupass"
 
+    envvars = [
+        "https_proxy",
+        "http_proxy",
+        "ftp_proxy",
+        "no_proxy",
+    ]
+
     # The script to run in the guest that builds QEMU
     BUILD_SCRIPT = ""
     # The guest name, to be overridden by subclasses
@@ -106,6 +113,8 @@  class BaseVM(object):
                    "-o", "UserKnownHostsFile=" + os.devnull,
                    "-o", "ConnectTimeout=1",
                    "-p", self.ssh_port, "-i", self._ssh_key_file]
+        for var in self.envvars:
+            ssh_cmd += ['-o', "SendEnv=%s" % var ]
         if interactive:
             ssh_cmd += ['-t']
         assert not isinstance(cmd, str)