Project

Profile

Help

Issue #4810

"force_full" should disable fast-forward instead of enabling

Added by Zhiming 9 months ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
2.20.0
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

It doesn't make sense that "force_full" enable fast-forward, so change it to be disable fast-forward.


Related issues

Related to Pulp - Issue #4799: Replace option "skip_fast_forward" with "force_full" for FileDistributor CLOSED - CURRENTRELEASE Actions

Associated revisions

Revision d60bd2ea View on GitHub
Added by Zhiming 8 months ago

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

History

#1 Updated by daviddavis 9 months ago

  • Tags Pulp 2 added

#2 Updated by dkliban@redhat.com 8 months ago

  • Triaged changed from No to Yes

#3 Updated by ttereshc 7 months ago

  • Status changed from NEW to MODIFIED
  • Platform Release set to 2.20.0

#4 Updated by ttereshc 7 months ago

  • Sprint/Milestone set to 2.20.0

#5 Updated by bherring 7 months ago

  • Related to Issue #4799: Replace option "skip_fast_forward" with "force_full" for FileDistributor added

#6 Updated by ttereshc 7 months ago

  • Status changed from MODIFIED to ON_QA

#7 Updated by ttereshc 6 months ago

  • Status changed from ON_QA to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF