Project

Profile

Help

Issue #4946

closed

pulp_content_host default value is set in the wrong ansible role

Added by jsherril@redhat.com almost 5 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
Installer - Moved to GitHub issues
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 54
Quarter:

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 definedCLOSED - CURRENTRELEASEbmbouterActions
Related to Pulp - Task #4964: Add variable to specify bind port for pulp-api and pulp-content-appCLOSED - CURRENTRELEASEbmbouter

Actions
Actions #1

Updated by jsherril@redhat.com almost 5 years ago

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

Updated by amacdona@redhat.com almost 5 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 54
Actions #3

Updated by amacdona@redhat.com almost 5 years ago

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

Updated by bmbouter almost 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
Actions #5

Updated by bmbouter almost 5 years ago

  • Status changed from ASSIGNED to POST

Added by bmbouter almost 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.

https://pulp.plan.io/issues/4963 closes #4963

https://pulp.plan.io/issues/4946 closes #4946

Added by Brian Bouterse almost 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.

https://pulp.plan.io/issues/4963 closes #4963

https://pulp.plan.io/issues/4946 closes #4946

Actions #6

Updated by bmbouter almost 5 years ago

  • Status changed from POST to MODIFIED
Actions #7

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #8

Updated by bmbouter almost 4 years ago

  • Category set to Installer - Moved to GitHub issues
  • Tags deleted (Pulp 3 installer)

Also available in: Atom PDF