zuul
/
zuul
Code Issues Proposed changes

Strip not split the untrusted_wrapper 

If we split the string then we end up with a list, which will never work
when looking up a driver. I assume this was supposed to have been strip
initially.

Change-Id: Ib6b3990743981853c8369ee6db0ef606fca3a5f0
Signed-off-by: Jamie Lennox <jamielennox@gmail.com>
This commit is contained in:
Jamie Lennox 2017-06-08 15:02:23 +10:00
parent fcb00c5c6a
commit 89785b93b2
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
zuul/executor/server.py
View File

@ -395,7 +395,7 @@ class ExecutorServer(object):
if self.config.has_option('executor', 'untrusted_wrapper'):
untrusted_wrapper_name = self.config.get(
'executor', 'untrusted_wrapper').split()
'executor', 'untrusted_wrapper').strip()
else:
untrusted_wrapper_name = 'bubblewrap'
self.untrusted_wrapper = connections.drivers[untrusted_wrapper_name]