search

pulp / Pulp-2-Tests

:warning: ⛔️ Pulp2 is EOL as of November 30 2022, for more info visit this link https://pulpproject.org/2022/09/19/pulp-2-eol/. ⛔️ Functional tests for Pulp 2.
https://pulp-2-tests.readthedocs.io/
1 stars 11 forks source link

Adding verification for skipping zchunks in repodata on sync. #186

Closed bherrin3 closed 5 years ago

bherrin3 commented 5 years ago

Problem

The inclusion of #4529 accounts for zchunks archives existing in repodata as .zck files. The feature was added to Fedora 30 with patches to createrepo_c --zck.

At this time, there are no repos that have purely .zck archives nor was that a consideration in the feature update.

Solution

Created a test that:

  1. Checks to ensure the development ticket has been implemented and at release 2.19.1 or greater
  2. Cleans any orphans as the test is searching on the file system for units by extension
  3. Sync a fedorapeople repo, rpm-zchunk, that has been created with createrepo_c --zck on F30. The fixture will be automated to rsync in pulp-fixtures once the base VM is updated to F30 and the fixture can be generated.
  4. Verify there is not a .zck unit on the system.

References

For more infomation on zchunks or the history for this update, see:

closes #4530

bherrin3 commented 5 years ago

Good input. I got pulled off onto other things.

Will save my local, push and re-open later once I have integrated the additional verification.

bherrin3 commented 5 years ago

Back into this task. Re-opening with all updates, constants added and additional verification, @kersommoura @rochacbruno