summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTill Riedel <riedel@teco.edu>2020-11-17 20:54:14 +0100
committerGitHub <noreply@github.com>2020-11-17 14:54:14 -0500
commita925b5a0ca4aa3e63b084c0f6664fe815c2c9db0 (patch)
treeb77aebe7b49e703c6d8724b679cb1cb28a2d7b12
parenteeef783b0322d99840f39a58de052772afefe822 (diff)
downloadvyos-cloud-init-a925b5a0ca4aa3e63b084c0f6664fe815c2c9db0.tar.gz
vyos-cloud-init-a925b5a0ca4aa3e63b084c0f6664fe815c2c9db0.zip
add --no-tty option to gpg (#669)
Make sure that gpg works even if the instance has no /dev/tty. This has been observed on Debian. LP: #1813396
-rw-r--r--cloudinit/gpg.py2
-rw-r--r--cloudinit/tests/test_gpg.py3
-rw-r--r--tools/.github-cla-signers1
3 files changed, 4 insertions, 2 deletions
diff --git a/cloudinit/gpg.py b/cloudinit/gpg.py
index be0ca0ea..3780326c 100644
--- a/cloudinit/gpg.py
+++ b/cloudinit/gpg.py
@@ -42,7 +42,7 @@ def recv_key(key, keyserver, retries=(1, 1)):
@param retries: an iterable of sleep lengths for retries.
Use None to indicate no retries."""
LOG.debug("Importing key '%s' from keyserver '%s'", key, keyserver)
- cmd = ["gpg", "--keyserver=%s" % keyserver, "--recv-keys", key]
+ cmd = ["gpg", "--no-tty", "--keyserver=%s" % keyserver, "--recv-keys", key]
if retries is None:
retries = []
trynum = 0
diff --git a/cloudinit/tests/test_gpg.py b/cloudinit/tests/test_gpg.py
index f96f5372..311dfad6 100644
--- a/cloudinit/tests/test_gpg.py
+++ b/cloudinit/tests/test_gpg.py
@@ -49,6 +49,7 @@ class TestReceiveKeys(CiTestCase):
m_subp.return_value = ('', '')
gpg.recv_key(key, keyserver, retries=retries)
m_subp.assert_called_once_with(
- ['gpg', '--keyserver=%s' % keyserver, '--recv-keys', key],
+ ['gpg', '--no-tty',
+ '--keyserver=%s' % keyserver, '--recv-keys', key],
capture=True)
m_sleep.assert_not_called()
diff --git a/tools/.github-cla-signers b/tools/.github-cla-signers
index 2090dc2a..b2464768 100644
--- a/tools/.github-cla-signers
+++ b/tools/.github-cla-signers
@@ -21,6 +21,7 @@ matthewruffell
nishigori
omBratteng
onitake
+riedel
slyon
smoser
sshedi