All jobs that use the old mock_genconfig fail following http_proxy env var addition

Description

09:09:46 #### Generating mock configuration
09:09:46 + ./mock_genconfig --name=epel-7-x86_64-ovirt-snapshot --base=epel-7-x86_64.cfg --option=basedir=/home/jenkins/workspace/ovirt-engine-extension-aaa-ldap_master_create-rpms-el7-x86_64_created/mock/ --try-proxy --repo=ovirt-master-snapshot,http://resources.ovirt.org/pub/ovirt-master-snapshot/rpm/el7 --repo=ovirt-master-snapshot-static,http://resources.ovirt.org/pub/ovirt-master-snapshot-static/rpm/el7
09:09:47 Traceback (most recent call last):
09:09:47 File "./mock_genconfig", line 161, in <module>
09:09:47 main()
09:09:47 File "./mock_genconfig", line 138, in main
09:09:47 execfile(base_config)
09:09:47 File "epel-7-x86_64_proxied.cfg", line 8, in <module>
09:09:47 config_opts['environment']['http_proxy'] = 'http://proxy.phx.ovirt.org:3128'
09:09:47 KeyError: 'environment'
09:09:47 Build step 'Execute shell' marked build as failure

http://jenkins.ovirt.org/job/ovirt-engine-extension-aaa-ldap_master_create-rpms-el7-x86_64_created/77/console

Activity

Show:

Barak Korren December 15, 2016 at 3:31 PM

Seems issue was fixed

Ondra Machacek December 15, 2016 at 2:22 PM

Works great, thanks a lot !

Barak Korren December 15, 2016 at 2:13 PM

Patch just merged.
Please confirm your jobs stop failing.

Barak Korren December 15, 2016 at 8:02 AM
Edited

This aught to fix remaining issues:
https://gerrit.ovirt.org/#/c/68503

Barak Korren December 14, 2016 at 5:17 PM

Bleh... right I should've known this was not gonna be that easy to fix...

Back to the drawing board with this...

And again I urge you to just move the standard CI, the code we have there is far more sane...

Fixed

Details

Assignee

Reporter

Components

Priority

Created December 14, 2016 at 9:12 AM
Updated May 25, 2017 at 11:31 AM
Resolved December 15, 2016 at 3:31 PM