Project

Profile

Help

Task #4964

closed

Task #4961: [Epic] Improve Integration and Testing with Nginx and Apache

Add variable to specify bind port for pulp-api and pulp-content-app

Added by bmbouter almost 5 years ago. Updated almost 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
Installer - Moved to GitHub issues
Sprint/Milestone:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 54
Quarter:

Description

The user should be able to specify the bind string for both pulp-api and/or pulp-content-app.


Related issues

Related to Pulp - Issue #4946: pulp_content_host default value is set in the wrong ansible roleCLOSED - CURRENTRELEASEbmbouterActions
Actions #1

Updated by bmbouter almost 5 years ago

  • Parent issue set to #4961
Actions #2

Updated by amacdona@redhat.com almost 5 years ago

  • Related to Issue #4946: pulp_content_host default value is set in the wrong ansible role added
Actions #3

Updated by amacdona@redhat.com almost 5 years ago

+1. Currently, we have:

pulp role:
pulp_api_host: localhost
pulp_api_port: 24817

pulp-content role:
pulp_content_host: 'localhost:24816'
pulp_content_bind: '0.0.0.0:24816'

I'm setting this to be related to https://pulp.plan.io/issues/4946 which should be completed at the same time.

Actions #4

Updated by mdepaulo@redhat.com almost 5 years ago

WRT the variables, they should be consistent across pulp-api & pulp-content.

Note that the pulp_api_host is used elsewhere in the ansible codebase, not just as the host to listen on. I think it gets advertised to clients.

So some refactoring should be done.

Actions #5

Updated by amacdona@redhat.com almost 5 years ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes
  • Sprint set to Sprint 54
Actions #6

Updated by bmbouter almost 5 years ago

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

Updated by bmbouter almost 5 years ago

  • Status changed from ASSIGNED to POST

Added by bmbouter almost 5 years ago

Revision 8eb135e6 | View on GitHub

Allow bind and nginx to be different

This introduces a pulp_api_bind variable in the 'pulp' playbook which the user can use to set the bind address (or multiple) of pulp-api. This is used by the pulp-api service template for the systemd service.

It moves the nginx and apache reverse proxy variables out of the 'pulp' playbook and into the 'pulp-webserver' playbook where they are used exclusively. The defaults are there also. It created 4 variables:

  • pulp_content_host
  • pulp_content_port
  • pulp_api_host
  • pulp_api_port

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

Added by Brian Bouterse almost 5 years ago

Revision 8eb135e6 | View on GitHub

Allow bind and nginx to be different

This introduces a pulp_api_bind variable in the 'pulp' playbook which the user can use to set the bind address (or multiple) of pulp-api. This is used by the pulp-api service template for the systemd service.

It moves the nginx and apache reverse proxy variables out of the 'pulp' playbook and into the 'pulp-webserver' playbook where they are used exclusively. The defaults are there also. It created 4 variables:

  • pulp_content_host
  • pulp_content_port
  • pulp_api_host
  • pulp_api_port

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

Actions #8

Updated by bmbouter almost 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #9

Updated by bmbouter over 4 years ago

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

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