David Caro has uploaded a new change for review.

Change subject: Fixed issue with non-existing config option
......................................................................

Fixed issue with non-existing config option

When using a mock configuration wothout yum.conf option, it will fail on
missing key.

Change-Id: Ie7d79734dfb8da64dc1518e8fab4c9a10a2f9f39
Signed-off-by: David Caro <dcaro...@redhat.com>
---
M mock_configs/mock_genconfig
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/jenkins refs/changes/81/27381/1

diff --git a/mock_configs/mock_genconfig b/mock_configs/mock_genconfig
index a6dbd49..9d2068a 100755
--- a/mock_configs/mock_genconfig
+++ b/mock_configs/mock_genconfig
@@ -85,7 +85,7 @@
         config_opts[key] = val
 
     # add the extra repos passed
-    config_opts['yum.conf'] = '%s' % (config_opts['yum.conf']
+    config_opts['yum.conf'] = '%s' % (config_opts.get('yum.conf', '')
                                       + get_repos(args.extra_repos))
 
     print dumps_dict(config_opts, 'config_opts')


-- 
To view, visit http://gerrit.ovirt.org/27381
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie7d79734dfb8da64dc1518e8fab4c9a10a2f9f39
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaro...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to