Project

Profile

Help

Issue #4529

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

Added by ttereshc 8 months ago. Updated 5 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
2.19.1
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
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 metadata CLOSED - COMPLETE Actions

Associated revisions

Revision 1a4cc6ab View on GitHub
Added by ttereshc 7 months 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 6 months 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 8 months ago

  • Description updated (diff)

#2 Updated by ttereshc 8 months ago

  • Description updated (diff)

#3 Updated by ttereshc 8 months ago

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

#4 Updated by ttereshc 8 months 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 8 months ago

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

#6 Updated by rchan 8 months ago

  • Sprint changed from Sprint 50 to Sprint 51

#7 Updated by bmbouter 7 months ago

  • Tags Pulp 2 added

#8 Updated by daviddavis 7 months ago

  • Sprint changed from Sprint 51 to Sprint 52

#9 Updated by dkliban@redhat.com 7 months ago

  • Platform Release set to 2.19.1

#10 Updated by ttereshc 7 months ago

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

#11 Updated by ttereshc 7 months ago

  • Status changed from ASSIGNED to POST

#12 Updated by dkliban@redhat.com 7 months ago

  • Sprint/Milestone set to 2.19.1

#13 Updated by ttereshc 6 months ago

  • Status changed from POST to MODIFIED

#14 Updated by ttereshc 6 months ago

#15 Updated by dkliban@redhat.com 6 months ago

  • Status changed from MODIFIED to ON_QA

#16 Updated by dkliban@redhat.com 6 months ago

  • Status changed from ON_QA to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF