Project

Profile

Help

Issue #4529

closed

Broken repo can be published if zchunk metadata is not ignored at sync time

Added by ttereshc about 5 years ago. Updated almost 5 years ago.

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

Description

In Fedora 30, Zchunk metadata is going to be present alongside with older compression formats.

Pulp should ignore and not sync this metadata, in order to avoid creating a potentially bad repo at publish time.
Scenario for a bad repo:
- create repo and sync from a remote repo with zchunk metadata
- make any content changes to a repo , e.g. remove/add packages
- published a repo
- zchunk metadata is published as-is without any modifications to it (even though content in a repo changed) -> clients which use zchunk metadata will get incorrect data about the content in a repo.

Repo with this new metadata type: http://mirror.vutbr.cz/fedora/development/rawhide/Everything/x86_64/os/


Related issues

Related to RPM Support - Test #4530: Test support of zchunk metadataCLOSED - COMPLETEbherringActions
Actions #1

Updated by ttereshc about 5 years ago

  • Description updated (diff)
Actions #2

Updated by ttereshc about 5 years ago

  • Description updated (diff)
Actions #3

Updated by ttereshc about 5 years ago

  • Related to Test #4530: Test support of zchunk metadata added
Actions #4

Updated by ttereshc about 5 years ago

  • Tracker changed from Story to Issue
  • Subject changed from As a user, zchunk metadata is supported to Broken repo can be published if zchunk metadata is not ignored at sync time
  • Description updated (diff)
  • Severity set to 2. Medium
  • Triaged set to No
Actions #5

Updated by ttereshc about 5 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 50
Actions #6

Updated by rchan about 5 years ago

  • Sprint changed from Sprint 50 to Sprint 51
Actions #7

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added
Actions #8

Updated by daviddavis about 5 years ago

  • Sprint changed from Sprint 51 to Sprint 52
Actions #9

Updated by dkliban@redhat.com almost 5 years ago

  • Platform Release set to 2.19.1
Actions #10

Updated by ttereshc almost 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ttereshc

Added by ttereshc almost 5 years ago

Revision 1a4cc6ab | View on GitHub

Skip zchunk metadata at sync time

Pulp 2 doesn't support zchunk compression format. Keeping mirrored zchunk metadata in a repository can cause a repository to be in a broken state.

closes #4529 https://pulp.plan.io/issues/4529

Actions #11

Updated by ttereshc almost 5 years ago

  • Status changed from ASSIGNED to POST
Actions #12

Updated by dkliban@redhat.com almost 5 years ago

  • Sprint/Milestone set to 2.19.1
Actions #13

Updated by ttereshc almost 5 years ago

  • Status changed from POST to MODIFIED

Added by ttereshc almost 5 years ago

Revision 0813de09 | View on GitHub

Skip zchunk metadata at sync time

Pulp 2 doesn't support zchunk compression format. Keeping mirrored zchunk metadata in a repository can cause a repository to be in a broken state.

closes #4529 https://pulp.plan.io/issues/4529

(cherry picked from commit 1a4cc6abffb632b4cc688e3540e9c009cce1da52)

Actions #14

Updated by ttereshc almost 5 years ago

Actions #15

Updated by dkliban@redhat.com almost 5 years ago

  • Status changed from MODIFIED to 5
Actions #16

Updated by dkliban@redhat.com almost 5 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

Also available in: Atom PDF