Project

Profile

Help

Issue #4810

"force_full" should disable fast-forward instead of enabling

Added by Zhiming about 1 year ago. Updated about 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
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

Related to Pulp - Issue #4799: Replace option "skip_fast_forward" with "force_full" for FileDistributorCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision d60bd2ea View on GitHub
Added by Zhiming about 1 year 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 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

Also available in: Atom PDF