Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #6050

closed

Sync fails if remote has no Suite

Added by quba42 almost 5 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
quba42
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...

Actions
Copy link
 #1

Updated by mdellweg almost 5 years ago

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

Actions
Copy link
 #2

Updated by quba42 over 4 years ago

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

Actions
Copy link
 #4

Updated by quba42 over 4 years ago

  • Status changed from NEW to POST
Actions
Copy link
 #5

Updated by quba42 over 4 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #6

Updated by quba42 over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF