Project

Profile

Help

Issue #6050

Sync fails if remote has no Suite

Added by quba42 9 months ago. Updated 7 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version - Debian:
master
Platform Release:
Target Release - Debian:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

pulp_deb currently assumes that the "Suite" field in Release files always exists.

The following suggests that a "Suite" field is optional in Release files:
https://wiki.debian.org/DebianRepository/Format#Suite

Repositories without a "Suite" field exist in the wild:
http://deb.tuxedocomputers.com/ubuntu/dists/bionic/

Attempting to sync such repositories causes an error:

pulp: rq.worker:ERROR: KeyError: 'Suite'

Even if the KeyError is fixed, some fall back value must be provided since "Suite" is a mandatory DB field in pulp_deb.

Option 1: Find some suitable default fall back value when a remote has no "Suite" field in it's Release file.
Option 2: Do not make the DB field mandatory.

Option 1 is very easy to implement, option 2 would require migrations and could mess up the plugin logic...

Associated revisions

History

#1 Updated by mdellweg 9 months ago

If it's going to be Option 2, it is the right time now to make backward incompatible database changes.

#2 Updated by quba42 7 months ago

I may have been mistaken about the field being mandatory... Will add a PR.

#4 Updated by quba42 7 months ago

  • Status changed from NEW to POST

#5 Updated by quba42 7 months ago

  • Status changed from POST to MODIFIED

Please register to edit this issue

Also available in: Atom PDF