Project

Profile

Help

Issue #4529

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

Added by ttereshc over 1 year ago. Updated about 1 year 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

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 1 year 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 about 1 year 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 1 year ago

  • Description updated (diff)

#2 Updated by ttereshc over 1 year ago

  • Description updated (diff)

#3 Updated by ttereshc over 1 year ago

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

#4 Updated by ttereshc over 1 year 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 1 year ago

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

#6 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 50 to Sprint 51

#7 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

#8 Updated by daviddavis over 1 year ago

  • Sprint changed from Sprint 51 to Sprint 52

#9 Updated by dkliban@redhat.com over 1 year ago

  • Platform Release set to 2.19.1

#10 Updated by ttereshc over 1 year ago

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

#11 Updated by ttereshc over 1 year ago

  • Status changed from ASSIGNED to POST

#12 Updated by dkliban@redhat.com over 1 year ago

  • Sprint/Milestone set to 2.19.1

#13 Updated by ttereshc over 1 year ago

  • Status changed from POST to MODIFIED

#14 Updated by ttereshc about 1 year ago

#15 Updated by dkliban@redhat.com about 1 year ago

  • Status changed from MODIFIED to 5

#16 Updated by dkliban@redhat.com about 1 year ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF