Issue #4810
"force_full" should disable fast-forward instead of enabling
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.20.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Description
It doesn't make sense that "force_full" enable fast-forward, so change it to be disable fast-forward.
Related issues
Associated revisions
History
#1
Updated by daviddavis about 1 year ago
- Tags Pulp 2 added
#2
Updated by dkliban@redhat.com about 1 year ago
- Triaged changed from No to Yes
#3
Updated by ttereshc about 1 year ago
- Status changed from NEW to MODIFIED
- Platform Release set to 2.20.0
#4
Updated by ttereshc about 1 year ago
- Sprint/Milestone set to 2.20.0
#5
Updated by bherring about 1 year ago
- Related to Issue #4799: Replace option "skip_fast_forward" with "force_full" for FileDistributor added
#6
Updated by ttereshc about 1 year ago
- Status changed from MODIFIED to 5
#7
Updated by ttereshc about 1 year ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
Please register to edit this issue
.
It doesn't make sense that "force_full" enable fast-forward, so change the opposite logic and enable fast-forward by default
ref #4810 https://pulp.plan.io/issues/4810
Fixing failed unit tests
Fixing failed unit tests and adding five unit tests for the method "publish_repo_fast_forward".
Besides that, improves "publish_repo_fast_forward" to delete symlinks that pointing files were removed, and the change also is covered by new unit tests.
Fixing PEP 8 issues