Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When the circumstances are just right, pulp_deb will produce inconsistent verbatim publications #549

Closed
quba42 opened this issue Jun 20, 2022 · 0 comments · Fixed by #550
Labels
.bugfix CHANGES/<issue_number>.bugfix

Comments

@quba42
Copy link
Collaborator

quba42 commented Jun 20, 2022

Version
Affects every version tested (at least the cutting edge all the way back to 3.14)

Describe the bug
Does not happen with download policy immediate, (reproduced with streamed, but on_demand is probably the same).

When re-syncing a repo with a unchanged Package index file, but a changed associated Package.gz file. (Can happen when syncing from another Pulp instance that uses a new APT publication with the same repo version.), then the existing PackageIndex is given a new artifact, rather than creating a new PackageIndex content. As a result existing repo versions have a release file referencing a Package.gz file with a different checksum than the one now included. When verbatim publishing such a repo versions clients will rightly complain about failed checksums.

Expected behavior
Metadata content should be completely immutable. If anything changes new metadata should be created, not existing content altered!

Additional context
The problem is the uniqueness constraints on the affected content types!

@quba42 quba42 added Triage-Needed .bugfix CHANGES/<issue_number>.bugfix and removed Triage-Needed labels Jun 20, 2022
quba42 added a commit to ATIX-AG/pulp_deb that referenced this issue Jun 20, 2022
quba42 added a commit to ATIX-AG/pulp_deb that referenced this issue Jun 21, 2022
quba42 added a commit to ATIX-AG/pulp_deb that referenced this issue Jun 21, 2022
quba42 added a commit to ATIX-AG/pulp_deb that referenced this issue Jun 21, 2022
quba42 added a commit to ATIX-AG/pulp_deb that referenced this issue Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.bugfix CHANGES/<issue_number>.bugfix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant