Ewoud Kohl van Wijngaarden has posted comments on this change. Change subject: It releasever not set it adds centos to the distro ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/29354/2/mock_configs/epel-7-x86_64.cfg File mock_configs/epel-7-x86_64.cfg: Line 2: config_opts['target_arch'] = 'x86_64' Line 3: config_opts['legal_host_arches'] = ('x86_64',) Line 4: config_opts['chroot_setup_cmd'] = 'install @buildsys-build' Line 5: config_opts['dist'] = 'el7' # only useful for --resultdir variable subst Line 6: config_opts["releasever"]="7" This may be nitpicking, but the rest is ' = ' instead of '='. Line 7: Line 8: config_opts['yum.conf'] = """ Line 9: [main] Line 10: cachedir=/var/cache/yum -- To view, visit http://gerrit.ovirt.org/29354 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I856148c384b1ffc60a571a1e821d899ede3f89bf Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
