Conversation
9a5c3b4 to
123dae0
Compare
|
@Moustafa-Moustafa Can you rebase with changes from main? That should fix the CI issues. |
PublishedMetadata.create_from_file() unconditionally saved artifacts to storage on every publish. When a publication was created for an already-published repository version, this overwrote identical metadata blobs with new timestamps, causing problems with CDN caching layers that use last-modified times. Applied the same deduplication pattern used for content artifact uploads: look up the artifact by checksum first, verify the file exists in storage, and skip the write if it does. Assisted-by: GitHub Copilot closes pulp#7344
123dae0 to
46c4cfc
Compare
Thanks, Done. |
Backport to 3.63: 💚 backport PR created✅ Backport PR branch: Backported as #7460 🤖 @patchback |
Backport to 3.49: 💚 backport PR created✅ Backport PR branch: Backported as #7462 🤖 @patchback |
Backport to 3.85: 💚 backport PR created✅ Backport PR branch: Backported as #7459 🤖 @patchback |
Backport to 3.105: 💚 backport PR created✅ Backport PR branch: Backported as #7461 🤖 @patchback |
Backport to 3.73: 💚 backport PR created✅ Backport PR branch: Backported as #7463 🤖 @patchback |
Backport to 3.73: 💔 creation of the backport PR failed❌ Backport PR branch: Validation Failed: 'A pull request already exists for pulp:patchback/backports/3.73/862e1c2209d7a5efb30d3d585b362c4ef5e5b3cc/pr-7346.' Backporting merged PR #7346 into main
🤖 @patchback |
PublishedMetadata.create_from_file() unconditionally saved artifacts to storage on every publish. When a publication was created for an already-published repository version, this overwrote identical metadata blobs with new timestamps, causing problems with CDN caching layers that use last-modified times.
Applied the same deduplication pattern used for content artifact uploads: look up the artifact by checksum first, verify the file exists in storage, and skip the write if it does.
closes #7344
📜 Checklist
See: Pull Request Walkthrough