Project

Profile

Help

Issue #5767

ansible-pulp fails due to undefined pulp_settings_file when its roles are dynamically included

Added by mdepaulo@redhat.com 9 months ago. Updated 3 months ago.

Status:
NEW
Priority:
Normal
Assignee:
-
Category:
Installer
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
CI/CD
Sprint:

Description

This issue was caused by the introduction of the vars pulp_settings_file and pulp_config_dir to the pulp role (with default values), but it being used in other roles as well.

[The previous solution](https://github.com/pulp/ansible-pulp/pull/183) (now closed w/o merge) to this problem was to declare static dependencies of the other roles on the pulp role, but this means that if the roles are dynamically called, the pulp role will be re-run even more times.

I now believe the better solution is to add the 2 variables to the "defaults" of all roles that use it. So that it is a loosely coupled variable/role. We do this already for some other variables (pulp_user & pulp_install_dir).

History

#1 Updated by mdepaulo@redhat.com 9 months ago

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

#2 Updated by fao89 9 months ago

  • Triaged changed from No to Yes

#3 Updated by mdepaulo@redhat.com 9 months ago

  • Subject changed from Ansible roles fail to evaluate pulp_settings_file when dynamically included to ansible-pulp fails due to undefined pulp_settings_file when its roles are dynamically included

#4 Updated by fao89 8 months ago

  • Tags CI/CD added

#5 Updated by bmbouter 3 months ago

  • Category set to Installer
  • Tags deleted (Pulp 3 installer)

#6 Updated by dkliban@redhat.com 3 months ago

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

Please register to edit this issue

Also available in: Atom PDF