modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-05-16 13:56:11 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Only use default modules that were built with the same base module

Browse Source
This commit is contained in:
mprahl
2019-10-08 11:38:39 -04:00
parent 339c1ef98d
commit 27f4d71392
2 changed files with 14 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
module_build_service/scheduler/default_modules.py
View File

@@ -84,22 +84,23 @@ def add_default_modules(db_session, mmd, arches):
)
continue
try:
# We are reusing resolve_requires instead of directly querying the database since it
# provides the exact format that is needed for mbs.xmd.buildrequires.
#
# Only one default module is processed at a time in resolve_requires so that we
# are aware of which modules are not in the database, and can add those that are as
# buildrequires.
resolver = GenericResolver.create(db_session, conf)
resolved = resolver.resolve_requires([ns])
except UnprocessableEntity:
# Query for the latest default module that was built against this base module
resolver = GenericResolver.create(db_session, conf)
default_module_mmds = resolver.get_buildrequired_modulemds(name, stream, bm_mmd)
if not default_module_mmds:
log.warning(
"The default module %s from %s is not in the database and couldn't be added as "
"a buildrequire",
ns, bm_nsvc,
)
continue
# Since a default module entry only has the name and stream, there's no way to know
# which context to pick from if multiple are present. In this case, just pick the first
# one, which is the latest version but potentially a random context.
default_module_mmd = default_module_mmds[0]
# Use resolve_requires since it provides the exact format that is needed for
# mbs.xmd.buildrequires
resolved = resolver.resolve_requires([default_module_mmd.get_nsvc()])
nsvc = ":".join([name, stream, resolved[name]["version"], resolved[name]["context"]])
log.info("Adding the default module %s as a buildrequire", nsvc)