Issue #4946
closed
pulp_content_host default value is set in the wrong ansible role
Description
the default value for pulp_content_host is set here: roles/pulp-content/defaults/main.yml
but its used in the 'pulp' role here: https://github.com/pulp/ansible-pulp/blob/master/roles/pulp/templates/settings.py.j2#L5-L7
It seems that the default will be used depending on the order that things are loaded, but its not guaranteed. The default and the docs for that variable should be moved to the pulp role.
Related issues
Updated by jsherril@redhat.com over 5 years ago
- Related to Issue #4945: Error thrown when fetching a task that created a distribution if CONTENT_HOST is not defined added
Updated by amacdona@redhat.com over 5 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 54
Updated by amacdona@redhat.com over 5 years ago
- Related to Task #4964: Add variable to specify bind port for pulp-api and pulp-content-app added
Updated by bmbouter over 5 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to bmbouter
Updated by bmbouter over 5 years ago
- Status changed from ASSIGNED to POST
PR available at: https://github.com/pulp/ansible-pulp/pull/122
Added by bmbouter over 5 years ago
Added by Brian Bouterse over 5 years ago
Revision 03374b0a | View on GitHub
Default to localhost bind and move var
The pulp_content_host was being defined in pulp-content, but being used in the 'pulp' role. This moves the setting to the pulp/README.md and removes it from being a default specified by main.yml.
Also the pulp_content_bind is now 127.0.0.1 instead of 0.0.0.0 as part of story 4963.
Updated by bmbouter over 5 years ago
- Status changed from POST to MODIFIED
Applied in changeset ansible-pulp3|03374b0adf9effb9ef9969de405dfa2670735c6b.
Updated by bmbouter almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Updated by bmbouter over 4 years ago
- Category set to Installer - Moved to GitHub issues
- Tags deleted (
Pulp 3 installer)
.
Default to localhost bind and move var
The pulp_content_host was being defined in pulp-content, but being used in the 'pulp' role. This moves the setting to the pulp/README.md and removes it from being a default specified by main.yml.
Also the pulp_content_bind is now 127.0.0.1 instead of 0.0.0.0 as part of story 4963.
https://pulp.plan.io/issues/4963 closes #4963
https://pulp.plan.io/issues/4946 closes #4946