From 5b4069aeb774650f39ca59de777a3694e29349d3 Mon Sep 17 00:00:00 2001 From: Andrii Ostapenko Date: Tue, 5 Mar 2019 13:31:09 -0600 Subject: [PATCH] Support empty ssh_key_filename from configuration Support the case when key is provided by ssh-agent, as there's no way to provide None value from config to StrOpt without changing default behavior. Change-Id: Ib715dbad5b9069f55320bce26182a19aeebf5201 --- stackalytics/processor/rcs.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/stackalytics/processor/rcs.py b/stackalytics/processor/rcs.py index 88e37791c..14ad9e289 100644 --- a/stackalytics/processor/rcs.py +++ b/stackalytics/processor/rcs.py @@ -80,7 +80,7 @@ class Gerrit(Rcs): self.close() def setup(self, **kwargs): - self.key_filename = kwargs.get('key_filename') + self.key_filename = kwargs.get('key_filename') or None self.username = kwargs.get('username') self.ssh_errors_limit = kwargs.get('gerrit_retry') or SSH_ERRORS_LIMIT