Project

Profile

Help

Issue #7886

closed

Issue #7673: Support synchronization of upstream repos with minimal Release file fields including for flat repos

Make Codename field optional

Added by quba42 almost 4 years ago. Updated almost 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version - Debian:
Platform Release:
Target Release - Debian:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

Right now we require upstream Release files to publish a "Codename" field.

Since all our internals actually use the "distribution" and not the codename (which is a path and therefore allways exists uniquely), we don't actually need repos to have a codename.

Examples of repositories without a codename exist in the wild, so we should make it optional. This will be easy to do.

Actions #1

Updated by quba42 almost 4 years ago

I am using the following reproducer/test case:

BASE_ADDR=':'
ENTITIES_NAME='test1'

REPO_HREF=$(http post "${BASE_ADDR}"/pulp/api/v3/repositories/deb/apt/ name="${ENTITIES_NAME}" | jq -r '.pulp_href')
REMOTE_HREF=$(http post "${BASE_ADDR}"/pulp/api/v3/remotes/deb/apt/ name="${ENTITIES_NAME}" distributions="bionic" components="stable" architectures="amd64" url="https://download.docker.com/linux/ubuntu/" | jq -r '.pulp_href')
SYNC_TASK_HREF=$(http post "${BASE_ADDR}${REPO_HREF}"sync/ remote="${REMOTE_HREF}" | jq -r '.task')
http get "${BASE_ADDR}${SYNC_TASK_HREF}"
Actions #2

Updated by pulpbot almost 4 years ago

  • Status changed from NEW to POST

Added by quba42 almost 4 years ago

Revision 587b766b | View on GitHub

Make codename an optional field

fixes #7886 https://pulp.plan.io/issues/7886

Actions #3

Updated by quba42 almost 4 years ago

  • Status changed from POST to MODIFIED
Actions #4

Updated by quba42 almost 4 years ago

  • Sprint/Milestone set to 2.8.0
Actions #5

Updated by pulpbot almost 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF