Project

Profile

Help

Issue #4529

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

Added by ttereshc over 2 years ago. Updated over 2 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 - COMPLETE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 1a4cc6ab View on GitHub
Added by ttereshc over 2 years ago

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

Revision 0813de09 View on GitHub
Added by ttereshc over 2 years ago

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)

History

#1 Updated by ttereshc over 2 years ago

  • Description updated (diff)

#2 Updated by ttereshc over 2 years ago

  • Description updated (diff)

#3 Updated by ttereshc over 2 years ago

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

#4 Updated by ttereshc over 2 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

#5 Updated by ttereshc over 2 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 50

#6 Updated by rchan over 2 years ago

  • Sprint changed from Sprint 50 to Sprint 51

#7 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

#8 Updated by daviddavis over 2 years ago

  • Sprint changed from Sprint 51 to Sprint 52

#9 Updated by dkliban@redhat.com over 2 years ago

  • Platform Release set to 2.19.1

#10 Updated by ttereshc over 2 years ago

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

#11 Updated by ttereshc over 2 years ago

  • Status changed from ASSIGNED to POST

#12 Updated by dkliban@redhat.com over 2 years ago

  • Sprint/Milestone set to 2.19.1

#13 Updated by ttereshc over 2 years ago

  • Status changed from POST to MODIFIED

#14 Updated by ttereshc over 2 years ago

#15 Updated by dkliban@redhat.com over 2 years ago

  • Status changed from MODIFIED to 5

#16 Updated by dkliban@redhat.com over 2 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF