Project

Profile

Help

Test #4269

closed

Recursive and conservative recursive copy

Added by milan about 6 years ago. Updated over 5 years ago.

Status:
CLOSED - COMPLETE
Priority:
Urgent
Assignee:
Sprint/Milestone:
-
Version:
Platform Release:
Tags:
Pulp 2
Sprint:
Quarter:

Description

Description

With the fix for the Issue #4152, the default behaviour of recursive copy changes back to the state before 2.17, rendering the test case CopyRecursiveUnitsTestCase broken.
The behaviour specified in the pulp smash issue 1090, as requested in the Issue #2478, is now optional and can be triggered by providing the "recursive_conservative": true config override instead of the default "recursive": true.
With that setting, the test case CopyRecursiveUnitsTestCase should agin pass as is.

Proposed solution

Split the test case CopyRecursiveUnitsTestCase such that it asserts:

  • a failure in the terms described in pulp smash issue 1090 with the default "recursive": true config override setting
  • a success once the non-default "recursive_conservative": true config override setting is provided

Related issues

Related to RPM Support - Issue #4152: Regression Pulp 2.17.1: recursive copy of RPMs does not copy partially resolvable dependenciesCLOSED - CURRENTRELEASEmilanActions
Related to Pulp - Test #4359: 2.18.1 TestingCLOSED - COMPLETEbherringActions
Related to RPM Support - Test #4543: Test newly described definitions of recursive and recursive_conservative flagsCLOSED - COMPLETEbherringActions
Actions #1

Updated by kersom about 6 years ago

  • Related to Issue #4152: Regression Pulp 2.17.1: recursive copy of RPMs does not copy partially resolvable dependencies added
Actions #2

Updated by kersom almost 6 years ago

  • Assignee set to kersom
Actions #3

Updated by bherring almost 6 years ago

  • Priority changed from Normal to High
Actions #4

Updated by bherring almost 6 years ago

  • Priority changed from High to Urgent
Actions #5

Updated by bherring almost 6 years ago

Actions #6

Updated by kersom almost 6 years ago

Tests were added to this one:

See: https://github.com/PulpQE/Pulp-2-Tests/pull/156

Actions #7

Updated by kersom almost 6 years ago

  • Status changed from NEW to ASSIGNED
Actions #8

Updated by kersom almost 6 years ago

  • Status changed from ASSIGNED to MODIFIED
Actions #9

Updated by kersom almost 6 years ago

  • Status changed from MODIFIED to CLOSED - COMPLETE
Actions #10

Updated by bherring almost 6 years ago

  • Related to Test #4543: Test newly described definitions of recursive and recursive_conservative flags added
Actions #11

Updated by bmbouter over 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF