Issue #4529: Broken repo can be published if zchunk metadata is not ignored at sync time - RPM Support - Pulp

Send by e-mail

Issue #4529

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

Added by ttereshc almost 2 years ago. Updated over 1 year ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

ttereshc

Sprint/Milestone:

Pulp - 2.19.1

Start date:

Due date:

Estimated time:

Severity:

2. Medium

Version:

Platform Release:

2.19.1

OS:

Triaged:

Yes

Groomed:

Sprint Candidate:

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

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 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

(cherry picked from commit 1a4cc6abffb632b4cc688e3540e9c009cce1da52)

History

#1 Updated by ttereshc almost 2 years ago

Description updated (diff)

#2 Updated by ttereshc almost 2 years ago

Description updated (diff)

#3 Updated by ttereshc almost 2 years ago

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

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