Project

Profile

Help

Issue #4946

pulp_content_host default value is set in the wrong ansible role

Added by jsherril@redhat.com 4 months ago. Updated 4 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 3 installer
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 54

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

Related to Pulp - Issue #4945: Error thrown when fetching a task that created a distribution if CONTENT_HOST is not defined MODIFIED Actions
Related to Pulp - Task #4964: Add variable to specify bind port for pulp-api and pulp-content-app MODIFIED Actions

Associated revisions

Revision 03374b0a View on GitHub
Added by bmbouter 4 months ago

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

History

#1 Updated by jsherril@redhat.com 4 months ago

  • Related to Issue #4945: Error thrown when fetching a task that created a distribution if CONTENT_HOST is not defined added

#2 Updated by amacdona@redhat.com 4 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 54

#3 Updated by amacdona@redhat.com 4 months ago

  • Related to Task #4964: Add variable to specify bind port for pulp-api and pulp-content-app added

#4 Updated by bmbouter 4 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter

#5 Updated by bmbouter 4 months ago

  • Status changed from ASSIGNED to POST

#6 Updated by bmbouter 4 months ago

  • Status changed from POST to MODIFIED

Please register to edit this issue

Also available in: Atom PDF