Project

Profile

Help

Test #4543

Test newly described definitions of recursive and recursive_conservative flags

Added by bherring 7 months ago. Updated 16 days ago.

Status:
CLOSED - COMPLETE
Priority:
High
Assignee:
Category:
-
Sprint/Milestone:
-
Version:
Platform Release:
Tags:
Documentation, Pulp 2
Sprint:

Description

Pulp 2.18.1 is introducing a new flag to `recursive_conservative`.

It is not clear in the documentation how this new flag should be used, and how to use this with already existent `recursive` flag.

I suggest this use case use to be added to the documentation.

Test Updates and Centralization on Modularity, depsolve, Recursive, and Recursive Conservative

Re-definition of Scope

There are a lot of modularity issues scattered about, quite a few holes in knowledge before this parent ticket, and a few missing test case permutations that need to be added.

Centralizing all this work and wrangling and straggling issues to this parent test issue.

The following are the actions being covered in this ticket:

test_modularity , #3740 --> #4364

PR: WIP, herring

  • Updating to include --recursive with latest RPMs
  • Updating to include --recursive_conservative cases
  • Updating to include modular <--> modular dep
    • There is already a fixture (rpm-with-modules) that has a module --> module --> RPM --> RPM dep tree
      • [duck-0.7] --> [walrus]
    • walrus modules then have a deptree of
      Modules:
      [walrus-0.71]
      └── walrus-0.71

      Dependent RPMs of the provided RPM from ``walrus-0.71``
      walrus-0.71
      └── whale
      ├── shark
      └── stork

      [walrus-5.21]
      └── walrus-5.21

test_copy.py

  • Update to include case that --recursive with latest RPMs

test_modular_errata.py

While not specifically addressed in #4364, modular_errata has been defined in the latest documentation update to behave the same for --recursive and --recursive_conservative as modules and modules with RPMs.

The test_modular_errata.py should be evaluated to handle this case and may need an update or new fixture for the errata testing.

  • PR: NEW
  • Updating to include --recursive with latest RPMs
  • Updating to include --recursive_conservative cases
  • Updating to include modular <--> modular dep

test_rich_weak_deps , #3740 --> #4547

  • PR: NEW
  • Ensure rich_weak_deps test includes --recursive and --recursive_conservative cases

Clean-up all Source Documentation

PR: DONE

  • Move any outstanding source documentation or issues relating to this work to this ticket
  • Close out and link any incorrectly created Git Issues to this ticket.
dev_baseline_meeting.log (2.5 KB) Raw Meeting notes bherring, 02/19/2019 04:53 PM dev_baseline_meeting.log

Checklist


Subtasks

Test #4364: Implement modularity content dependency solvingCLOSED - COMPLETEbherringActions
Test #4547: Ensure rich_weak_deps test includes --recursive and --recursive_conservative casesCLOSED - COMPLETEkersomActions
Test #4548: Ensure modular_errata.py includes cases for --recursive and --recursive_conservative inline with document updatesCLOSED - COMPLETEkersomActions

Related issues

Related to RPM Support - Test #4269: Recursive and conservative recursive copy CLOSED - COMPLETE Actions
Related to RPM Support - Story #3740: Implement modularity content dependency solving CLOSED - CURRENTRELEASE Actions
Copied from RPM Support - Task #4371: Document how to use the newly added recursive_conservative CLOSED - CURRENTRELEASE Actions

Associated revisions

Revision be4d1530 View on GitHub
Added by bherring 7 months ago

Updating CopyConservativeTestCase to include latest dependency test

Documentation added from #4371 highlighted that on
``--recursive``, the ``latest`` RPMs dependencies should be copied.

The creation of #4543 attempts to update tests due to the inclusion
of logic from #4371

Adding a case where ``walrus-0.71``RPM is already on B. Copying
the ``chimpanzee`` RPM will copy the latest version of ``walrus``
to B testing two copies of walrus available.

See:
- https://pulp.plan.io/issues/4371
- https://pulp.plan.io/issues/4543

refs #4543

History

#1 Updated by bherring 7 months ago

  • Copied from Task #4371: Document how to use the newly added recursive_conservative added

#2 Updated by bherring 7 months ago

  • Related to Test #4269: Recursive and conservative recursive copy added

#3 Updated by bherring 7 months ago

  • Related to Test #4364: Implement modularity content dependency solving added

#4 Updated by bherring 7 months ago

  • Related to Issue #3749: Skip upload of the foreign layers added

#5 Updated by bherring 7 months ago

  • Related to deleted (Issue #3749: Skip upload of the foreign layers)

#6 Updated by bherring 7 months ago

  • Related to Story #3740: Implement modularity content dependency solving added

#7 Updated by bherring 7 months ago

  • Status changed from NEW to ASSIGNED

#8 Updated by bherring 7 months ago

  • Related to Test #4158: Test applicability for modular or mixed content added

#9 Updated by bherring 7 months ago

  • Related to deleted (Test #4158: Test applicability for modular or mixed content)

#11 Updated by bherring 7 months ago

  • Priority changed from Normal to High

#12 Updated by bherring 7 months ago

  • Description updated (diff)

#13 Updated by bherring 7 months ago

  • Description updated (diff)

#14 Updated by bherring 7 months ago

  • Description updated (diff)

#15 Updated by bherring 7 months ago

  • Description updated (diff)

#16 Updated by bherring 7 months ago

  • Related to deleted (Test #4364: Implement modularity content dependency solving)

#17 Updated by bherring 7 months ago

  • Description updated (diff)

#18 Updated by bherring 7 months ago

  • Description updated (diff)

#19 Updated by bherring 7 months ago

  • Subject changed from Document how to use the newly added recursive_conservative to Test newly described definitions of recursive and recursive_conservative flags

#20 Updated by bherring 7 months ago

  • Checklist item rpm_copy for --recursive and --recursive_conservative with latest RPM definition added
  • Checklist item modular <-> modular copy with --recursive and --recursuve_conservative added
  • Checklist item modular <-> modular copy with RPM dependencies with --recursive and --recursive_conservative copy with latest RPM definiton added
  • Checklist item modular_errata copy with latests RPM definition added
  • Checklist item rich and weak dependencies with --recursive and --recursive_conservative with latest RPM definition added

#21 Updated by bherring 7 months ago

  • Checklist item rpm_errata copy for --recursive and --recursive_conservative with latest RPM definition added

#22 Updated by bherring 7 months ago

  • Sprint/Milestone set to 2.19.0

#23 Updated by bherring 7 months ago

  • Sprint/Milestone deleted (2.19.0)

#24 Updated by kersom 7 months ago

  • Checklist item rpm_copy for --recursive and --recursive_conservative with latest RPM definition set to Done

#25 Updated by bherring 7 months ago

  • Checklist item modular <-> modular copy with --recursive and --recursuve_conservative set to Done
  • Checklist item modular <-> modular copy with RPM dependencies with --recursive and --recursive_conservative copy with latest RPM definiton set to Done
  • Checklist item modular_errata copy with latests RPM definition set to Done
  • Checklist item rich and weak dependencies with --recursive and --recursive_conservative with latest RPM definition set to Done

#26 Updated by bherring 7 months ago

  • Checklist item deleted (rpm_errata copy for --recursive and --recursive_conservative with latest RPM definition)
  • Status changed from ASSIGNED to MODIFIED

Notes

Removed the checklist item for rpm_errata copy.

With all the permutations covered in modular and the module <--> module <-> RPM dependency list, that test was a superset of those specific permutations.

All Checklist items complete and PR'd. All tests currently running without error on 2.19.

#27 Updated by bmbouter 6 months ago

  • Tags Pulp 2 added

#28 Updated by bherring 5 months ago

  • Status changed from MODIFIED to CLOSED - COMPLETE

Please register to edit this issue

Also available in: Atom PDF