Fixes: https://pagure.io/koji/issue/3958
rebased onto d1c0d772909827323e1cb87bbfa7491dc984c9a4
More simple:
if self.config['extra'].get('mock.bootstrap_image_ready', False):
Not necessarily true - e.g. buildcfg['extra']['mock.bootstrap_image_ready'] = False In fact, next lines ('use_bootstrap') should be also fixed. People sometimes don't delete the extra record, just set it to False. So,
buildcfg['extra']['mock.bootstrap_image_ready'] = False
False
if 'mock.bootstrap_image_ready' in buildcfg['extra']: opts['bootstrap_image_ready'] = bool(buildcfg['extra']['mock.bootstrap_image_ready'] # no else part as it would be driven by site-defaults.cfg
This is not right place. It should be part of https://docs.pagure.org/koji/using_the_koji_build_system/#tuning-mock-s-behavior-per-tag Also, the next command is not true (container needs these packages, but not the builder)
rebased onto 310d087eca1856bbca9ef353b6310fa3c435a400
Metadata Update from @tkopecek: - Pull-request tagged with: testing-ready
Metadata Update from @tkopecek: - Pull-request untagged with: testing-ready - Pull-request tagged with: testing-basic
Metadata Update from @mfilip: - Pull-request tagged with: testing-done
Commit 8ad473b6 fixes this pull-request
Pull-Request has been merged by tkopecek
Fixes: https://pagure.io/koji/issue/3958