Skip to content

Commit

Permalink
setuptargetrepos - pesid repo fix
Browse files Browse the repository at this point in the history
  • Loading branch information
pirat89 committed Sep 11, 2021
1 parent fa8b25b commit 1b08aee
Showing 1 changed file with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ def process():
# Now get the info what should be the target RHEL repositories
expected_repos = repomap.get_expected_target_pesid_repos(enabled_repoids)
target_rhel_repoids = set()
for target_pesid, target_repoid in expected_repos.items():
if not target_repoid:
for target_pesid, target_pesidrepo in expected_repos.items():
if not target_pesidrepo:
# With the original repomap data, this should not happen (this should
# currently point to a problem in our data
# TODO(pstodulk): add report? inhibitor? what should be in the report?
Expand All @@ -102,10 +102,10 @@ def process():
.format(target_pesid)
)
continue
if target_repoid in excluded_repoids:
api.current_logger().debug('Skipping the {} repo (excluded).'.format(target_repoid))
if target_pesidrepo.repoid in excluded_repoids:
api.current_logger().debug('Skipping the {} repo (excluded).'.format(target_pesidrepo.repoid))
continue
target_rhel_repoids.add(target_repoid)
target_rhel_repoids.add(target_pesidrepo.repoid)

# FIXME: this could possibly result into a try to enable multiple repositories
# from the same family (pesid). But unless we have a bug in previous actors,
Expand Down

0 comments on commit 1b08aee

Please sign in to comment.