Issue #4094: Architecture is not created if there are no architecture specific packages - Debian Support - Pulp

Actions

Send by e-mail Copy link

Issue #4094

closed

Architecture is not created if there are no architecture specific packages

Added by quba42 about 4 years ago. Updated over 1 year ago.

Status:

CLOSED - WONTFIX

Priority:

High

Assignee:

Sprint/Milestone:

Start date:

Due date:

Estimated time:

Severity:

3. High

Version - Debian:

Platform Release:

Target Release - Debian:

OS:

Triaged:

Yes

Groomed:

Sprint Candidate:

Tags:

Performance, Pulp 2

Sprint:

Quarter:

Description

Current Behavior: Since the architectures supported by a repository are parsed from the packages list in the mongodb, architectures included in an upstream repository are not created if there are no architecture specific packages.

Expected Behavior: The architectures should be created containing only "architectures = all" type packages.

Proposed Approach:

There should be a units_deb_arches collection in the mongodb in addition to the existing units_deb_component, units_deb_release, and units_deb collections.

This would have at least two additional benefits beyond fixing this bug:

Parsing lists of all Packages to extract a list of one or two architectures is not good for performance.
It is conceptually correct. Debian repositories are built from a pool of all packages, along with meta data that will allow you to identify useful subsets of these packages using a combination of releases, components, and architectures. The database should mirror this conceptual division.

Related issues

Actions

Copy link

Updated by mihai.ibanescu@gmail.com about 4 years ago

That sounds reasonable.

I don't think I understand the use of architectures = all, so I'll need to do some research.

Actions

Copy link

Updated by quba42 about 4 years ago

I am currently working on refactoring the relevant code for better performance.
Once that is done I may look into fixing this bug as well (if I can find the time).

Actions

Copy link

Updated by quba42 about 4 years ago

The following pull request (https://github.com/pulp/pulp_deb/pull/57) now mentions this Bug. While this pull request does not fix the bug it lays the groundwork for doing so as mentioned above.

Actions

Copy link

Updated by quba42 about 4 years ago

mihai.ibanescu@gmail.com wrote:

I don't think I understand the use of architectures = all, so I'll need to do some research.

My understanding is that these are packages that are architecture independent packages (for example packages containing documentation) that can therefore be included with any architecture.

Actions

Copy link