Project

General

Profile

Bug #33358

RPM include filter includes extraneous packages from package groups having only type="default" packagereqs

Added by Laura Hild over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Content Views
Target version:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

At time of writing, with http://mirror.centos.org/centos-7/7.9.2009/ these are mainframe-access, networkmanager-submodules, internet-browser, and platform-microsoft, and with https://download.fedoraproject.org/pub/epel/7/x86_64/, milkymist.

Create a Content View, add EPEL7, add an include filter for some RPMs, publish and expect to get just those RPMs, but end up with the packages in @milkymist also.

What I believe to be happening: Katello includes package groups for which it determines the requirements are satisfied by the packages selected (filter_package_groups_by_pulp_href in app/lib/katello/util/pulpcore_content_filters.rb). The method which lists the requirements (package_names in app/models/katello/package_group.rb) produces it by combining the results of {default,optional,conditional,mandatory}_package_names in app/services/katello/pulp3/package_group.rb, whose mapping of the numeric identifiers is off by one because C numbers enums from 0. So package_names does not include type="default" packagereqs, and when a package group contains only type="default" packagereqs, package_names returns empty, is trivially satisfied, and the package group is included in the list of content to be copied by Pulp. Pulp, however, then recognizes that there are unsatisfied packagereqs and includes them.

Patch forthcoming.

Associated revisions

Revision 0d48c3ae (diff)
Added by Laura Hild over 1 year ago

Fixes #33358 - packagereq types off by 1

History

#1 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9582 added

#2 Updated by James Jeffers over 1 year ago

  • Bugzilla link set to 1959555

#3 Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases Katello 4.3.0 added

#4 Updated by Anonymous over 1 year ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Justin Sherrill over 1 year ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.2.0

#6 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/9656 added

Also available in: Atom PDF