Project

Profile

Help

Test #4738

File repositories shouldn't have duplicate files

Added by daviddavis 6 months ago. Updated 5 months ago.

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

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 repositories MODIFIED Actions
Related to Pulp - Test #4255: Checksum constraint error when syncing a package CLOSED - COMPLETE Actions

History

#1 Updated by daviddavis 6 months ago

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

#2 Updated by ragbalak 5 months ago

  • Assignee set to ragbalak

#3 Updated by ragbalak 5 months ago

  • Status changed from NEW to ASSIGNED

#4 Updated by kersom 5 months ago

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

#5 Updated by ragbalak 5 months 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 5 months ago

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

Please register to edit this issue

Also available in: Atom PDF