Project

Profile

Help

Test #4738

File repositories shouldn't have duplicate files

Added by daviddavis over 1 year ago. Updated over 1 year ago.

Status:
NEW
Priority:
Normal
Assignee:
-
Sprint/Milestone:
-
Platform Release:
Tags:
Sprint:
Quarter:

Description

File repositories shouldn't have content units that have the same relative_path. Files are written to the filesystem based on their relative paths so obviously only one file can be written.

Test case 1

1. Sync a file remote repo with a file (eg 1.iso)
2. Sync a second remote with a different file with the same name
3. Confirm that there aren't two files with the same in the repo

For step 1, if you don't have two remotes that have different files with the same name, you could upload file instead and then sync for step 2.

Test case 2

This is the reproducer in #4028.

1. Upload two files and create two file content units with the same filename (e.g. 1.iso)
2. Add both files to your repo
3. Confirm that there aren't two files with the same in the repo


Related issues

Related to File Support - Task #4028: Prevent duplicate files in repositoriesCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Related to Pulp - Test #4255: Checksum constraint error when syncing a packageCLOSED - COMPLETE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by daviddavis over 1 year ago

  • Related to Task #4028: Prevent duplicate files in repositories added

#2 Updated by ragbalak over 1 year ago

  • Assignee set to ragbalak

#3 Updated by ragbalak over 1 year ago

  • Status changed from NEW to ASSIGNED

#4 Updated by kersom over 1 year ago

  • Related to Test #4255: Checksum constraint error when syncing a package added

#5 Updated by ragbalak over 1 year ago

kersom bherring

The testcase(both) for those are added in the branch https://github.com/ragabala/pulp_file/commits/fix_4738.

The first testcase is being testerd and meged in https://github.com/pulp/pulp_file/pull/216

Thanks and Regards,
Ragavendran B

#6 Updated by ragbalak over 1 year ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (ragbalak)

Please register to edit this issue

Also available in: Atom PDF