Project

Profile

Help

Task #6071

closed

Story #5762: [Epic] CI Improvements

Task #5766: ansible-pulp CI should test calling roles from dynamic "include_role" in addition to static "roles"

ansible-pulp CI should test the calling roles from dynamic "include_role" with the source dirs of the branch

Added by mdepaulo@redhat.com almost 5 years ago. Updated over 4 years ago.

Status:
CLOSED - COMPLETE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
CI/CD
Sprint:
Quarter:

Description

As a follow up to #5766, we also need to test calling dynamic include_role with the source_dir, like in the "source" scenario.

Otherwise, the master branch never gets tested at all.

And soon (Story #5890), the latest 3.x branches will need to be tested as well.


Related issues

Related to Pulp - Story #5890: [Epic] The ansible-pulp installer needs to handle multiple versions of PulpCLOSED - WONTFIXmdepaulo@redhat.com

Actions
Actions #1

Updated by mdepaulo@redhat.com almost 5 years ago

  • Related to Story #5890: [Epic] The ansible-pulp installer needs to handle multiple versions of Pulp added
Actions #2

Updated by mdepaulo@redhat.com almost 5 years ago

  • Description updated (diff)
Actions #3

Updated by mdepaulo@redhat.com almost 5 years ago

  • Description updated (diff)
  • Tags CI/CD added

Added by Mike DePaulo almost 5 years ago

Revision 7e651869 | View on GitHub

ansible-pulp CI should test the calling roles from dynamic

"include_role" with the source dirs of the branch

Implementation includes:

  1. new source-dynamic scenario, using vars from example-source
  2. "dynamic" now uses same vars as in example-use (which facilitates documentation for users.)

fixes: #6071 https://pulp.plan.io/issues/6071

Added by Mike DePaulo almost 5 years ago

Revision 7e651869 | View on GitHub

ansible-pulp CI should test the calling roles from dynamic

"include_role" with the source dirs of the branch

Implementation includes:

  1. new source-dynamic scenario, using vars from example-source
  2. "dynamic" now uses same vars as in example-use (which facilitates documentation for users.)

fixes: #6071 https://pulp.plan.io/issues/6071

Actions #4

Updated by Anonymous almost 5 years ago

  • Status changed from NEW to MODIFIED
  • % Done changed from 0 to 100
Actions #5

Updated by dkliban@redhat.com over 4 years ago

  • Status changed from MODIFIED to CLOSED - COMPLETE

Also available in: Atom PDF